-
Notifications
You must be signed in to change notification settings - Fork 377
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
MSC2249: Require users to have visibility on an event when submitting reports #2249
Conversation
it's not in review if it's a WIP. |
This is no longer in WIP |
6322065
to
aa43896
Compare
Co-authored-by: Hubert Chathi <hubert@uhoreg.ca>
Given synapse has accidentally implemented this MSC, and it makes sense, we should just accept it. @mscbot fcp merge |
Team member @turt2live has proposed to merge this. The next step is review by the rest of the tagged people: Once at least 75% of reviewers approve (and there are no outstanding concerns), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for information about what commands tagged team members can give me. |
🔔 This is now entering its final comment period, as per the review above. 🔔 |
The final comment period, with a disposition to merge, as per the review above, is now complete. |
Merged 🎉 |
This fixes matrix-org/matrix-spec#525
Rendered
Implementation (by accident): matrix-org/synapse#13779
FCP tickyboxes