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.

Sign up for GitHub

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

Jump to bottom

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
@sfnuser

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.

@sfnuser sfnuser mentioned this issue Jun 28, 2024
Merged
@hdamker hdamker mentioned this issue Jul 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sfnuser sfnuser

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove unused messages object in SessionInfo schema sfnuser/QualityOnDemand
3 participants
@hdamker @eric-murray @sfnuser