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

Private rooms are now created with the .invited room history visibility #3853

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

Velin92
Copy link
Member

@Velin92 Velin92 commented Feb 28, 2025

fixes #3843

We want to always have private rooms to always start their room history with the .invited override

@Velin92 Velin92 requested a review from a team as a code owner February 28, 2025 09:57
@Velin92 Velin92 requested review from pixlwave and stefanceriu and removed request for a team February 28, 2025 09:57
@Velin92 Velin92 force-pushed the mauroromito/update_create_room_parameters_2 branch from 8135234 to e577f60 Compare February 28, 2025 09:58
@Velin92 Velin92 added the pr-bugfix for bug fix label Feb 28, 2025
@Velin92 Velin92 changed the title Private rooms are now created with the .default room history visibility Private rooms are now created with the .invited room history visibility Feb 28, 2025
Copy link

codecov bot commented Feb 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.96%. Comparing base (14fa7d8) to head (e7e39a2).
Report is 1 commits behind head on develop.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3853      +/-   ##
===========================================
- Coverage    78.96%   78.96%   -0.01%     
===========================================
  Files          794      794              
  Lines        70215    70215              
===========================================
- Hits         55445    55443       -2     
- Misses       14770    14772       +2     
Flag Coverage Δ
unittests 70.83% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

LGTM, what is the default value that this overrides out of interest?

@Velin92
Copy link
Member Author

Velin92 commented Feb 28, 2025

LGTM, what is the default value that this overrides out of interest?

the since selection option (or .shared)

@Velin92 Velin92 enabled auto-merge (squash) February 28, 2025 11:22
@Velin92 Velin92 merged commit 2b58f57 into develop Feb 28, 2025
8 checks passed
@Velin92 Velin92 deleted the mauroromito/update_create_room_parameters_2 branch February 28, 2025 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-bugfix for bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task] Set history visibility to "invited" when creating private room
2 participants