Skip to content

Conversation

@SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Jul 5, 2022

Fixes matrix-org/matrix-spec-proposals#3077 (comment) and makes the implementation more future-proof


Here's what your changelog entry will look like:

✨ Features

  • Add id to sdp_stream_metadata (#2493).

@SimonBrandner SimonBrandner requested a review from a team July 5, 2022 11:31
@SimonBrandner SimonBrandner requested a review from a team as a code owner July 5, 2022 11:31
@SimonBrandner SimonBrandner requested review from kerryarchibald, robintown and weeman1337 and removed request for kerryarchibald, robintown and weeman1337 July 5, 2022 11:31
@SimonBrandner SimonBrandner removed the request for review from a team July 5, 2022 12:30
Copy link
Contributor Author

@SimonBrandner SimonBrandner left a comment

Choose a reason for hiding this comment

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

Clearing review

@SimonBrandner SimonBrandner requested review from a team, kerryarchibald, robintown and weeman1337 and removed request for a team July 5, 2022 12:30
Copy link
Contributor

@weeman1337 weeman1337 left a comment

Choose a reason for hiding this comment

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

Is it possible to add tests for the changes?

@SimonBrandner SimonBrandner requested a review from weeman1337 July 5, 2022 17:13
Copy link
Contributor

@weeman1337 weeman1337 left a comment

Choose a reason for hiding this comment

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

Tests 🤗 LGTM

@SimonBrandner
Copy link
Contributor Author

Marking as draft until #2495 is merged

@SimonBrandner SimonBrandner marked this pull request as draft July 6, 2022 11:12
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner force-pushed the SimonBrandner/feat/sdp-meta-id branch from 434cb5f to 0ce0f7c Compare July 7, 2022 06:58
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner SimonBrandner force-pushed the SimonBrandner/feat/sdp-meta-id branch from 0ce0f7c to c85bf1b Compare July 7, 2022 07:24
@SimonBrandner
Copy link
Contributor Author

Replaced by #2551

@SimonBrandner SimonBrandner deleted the SimonBrandner/feat/sdp-meta-id branch July 31, 2022 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants