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

fix(api): Properly typed room-property updates - Part 2 #13173

Merged
merged 11 commits into from
Sep 11, 2024
Merged

fix(api): Properly typed room-property updates - Part 2 #13173

merged 11 commits into from
Sep 11, 2024

Conversation

nickvergessen
Copy link
Member

☑️ Resolves

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not possible
  • 📘 API documentation in docs/ has been updated or is not required
  • 🔖 Capability is added or not needed

@nickvergessen nickvergessen added 3. to review feature: api 🛠️ OCS API for conversations, chats and participants feature: conversations 👥 labels Aug 29, 2024
@nickvergessen nickvergessen added this to the 🖤 Next Major (31) milestone Aug 29, 2024
@nickvergessen nickvergessen self-assigned this Aug 29, 2024
@nickvergessen nickvergessen changed the title Techdebt/noid/unify room property updates 2 fix(api): Properly typed room-property updates - Part 2 Aug 29, 2024
@nickvergessen nickvergessen force-pushed the techdebt/noid/unify-room-property-updates-2 branch from 94dce3c to e8a0b09 Compare September 10, 2024 15:43
@nickvergessen nickvergessen force-pushed the techdebt/noid/unify-room-property-updates-2 branch from e8a0b09 to dee837f Compare September 11, 2024 07:03
@nickvergessen
Copy link
Member Author

Red CI fixed in #13256

@nickvergessen
fix(api): Properly typed callRecording update
d73e3e0 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed type update
7c12b93 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed readOnly update
55a3a71 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed listable update
4f82ee9 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed mentionPermissions update
ee42d44 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed description update
b113df8 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed password update
0206acd 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed messageExpiration update
428aa91 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed breakoutRoom update
38a244b 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(api): Properly typed avatar update
b13a01a 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
chore(assets): Recompile assets
f5e394f 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the techdebt/noid/unify-room-property-updates-2 branch from dee837f to f5e394f Compare September 11, 2024 07:45
@nickvergessen
Copy link
Member Author

/backport to stable30

@nickvergessen nickvergessen merged commit 507a6b5 into main Sep 11, 2024
69 checks passed
@nickvergessen nickvergessen deleted the techdebt/noid/unify-room-property-updates-2 branch September 11, 2024 08:20
@backportbot backportbot bot mentioned this pull request Sep 11, 2024
Merged
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanskar-soni-9 sanskar-soni-9 sanskar-soni-9 left review comments

@skalidindi53 skalidindi53 skalidindi53 approved these changes

Assignees

@nickvergessen nickvergessen

Labels
3. to review feature: api 🛠️ OCS API for conversations, chats and participants feature: conversations 👥
Projects
None yet
Milestone
🖤 Next Major (31)
Development

Successfully merging this pull request may close these issues.
3 participants
@nickvergessen @sanskar-soni-9 @skalidindi53