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

Remove draft status #236

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Remove draft status #236

wants to merge 2 commits into from

Conversation

aboba
Copy link
Collaborator

@aboba aboba commented Oct 21, 2023

The AV1 RTP payload specification is final, so it would be desirable to formally upgrade it from a draft to a final deliverable.

Fix for #231

Since the specification has been finalized, it is no longer a draft. 

Fix for #231
@aboba aboba changed the title Remote draft status Remove draft status Oct 21, 2023
@aboba aboba requested a review from mhoro October 21, 2023 21:47
@mhoro
Copy link
Collaborator

mhoro commented Oct 25, 2023

The proposed change looks good to me.
I'm not familiar with the git-hub review process, so let me know if there is some action, in addition to this comment, that is required of me as a reviewer.

Copy link

@dontcallmedom dontcallmedom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe the status line could be kept to say this is a "Final Deliverable"?

@aboba
Copy link
Collaborator Author

aboba commented Dec 13, 2023

@dontcallmedom I have updated the PR as you suggested.

@mhoro What are the next steps to close on this? Do we need to get approval of the AoMedia Board, or the RTC sub-group (or Codec WG)? Or can we just merge the PR?

@mhoro
Copy link
Collaborator

mhoro commented Dec 13, 2023 via email

@aboba
Copy link
Collaborator Author

aboba commented Mar 2, 2024

See answer from Adrian in #231. ETA is end of March 2024.

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 this pull request may close these issues.

3 participants