Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Purpose of messages array in SessionInfo schema is undocumented #309

Closed
sfnuser opened this issue Jun 21, 2024 · 2 comments · Fixed by #312
Closed

Purpose of messages array in SessionInfo schema is undocumented #309

sfnuser opened this issue Jun 21, 2024 · 2 comments · Fixed by #312
Assignees

Comments

@sfnuser
Copy link
Collaborator

sfnuser commented Jun 21, 2024

Problem description
SessionInfo schema in the spec contains messages object. The purpose of this object is undocumented

Expected behavior
This array object seems unnecessary and can be removed. If there is an purpose, it needs to be documented for implementation by vendors and use by developers.

@eric-murray
Copy link
Collaborator

Happy for this to be removed

@hdamker hdamker assigned hdamker and sfnuser and unassigned hdamker Jun 28, 2024
@hdamker
Copy link
Collaborator

hdamker commented Jun 28, 2024

There were no objections to remove the array object raised within the QoD Call today.
@sfnuser please create the PR for it as agreed.
@camaraproject/quality-on-demand_maintainers if you are using this array and want to keep it, please describe your use case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants