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

MSC1730: Mechanism for redirecting to an alternative server during login (version 2) #1748

Merged
merged 3 commits into from
Dec 17, 2018

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Dec 11, 2018

Rendered

Note: This is me re-opening #1730 because I accidentally screwed up the final comment period process and merged it too early. Please leave feedback here, and consider this MSC1730. I haven't given this one the proposal labels to avoid it being picked up as a whole new proposal - please see 1730 for the right labels.

See #1730 (comment) for more information.

@turt2live
Copy link
Member Author

@mscbot fcp merge

@mscbot
Copy link
Collaborator

mscbot commented Dec 11, 2018

Team member @turt2live has proposed to merge this. The next step is review by the rest of the tagged teams:

No concerns currently listed.

Once a majority of reviewers approve (and none object), 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 info about what commands tagged team members can give me.

@mscbot mscbot added proposed-final-comment-period Currently awaiting signoff of a majority of team members in order to enter the final comment period. disposition-merge labels Dec 11, 2018
@anoadragon453 anoadragon453 added proposal A matrix spec change proposal and removed proposal A matrix spec change proposal labels Dec 11, 2018
@richvdh
Copy link
Member

richvdh commented Dec 12, 2018

removing the labels since we're tracking this at #1730.

@richvdh richvdh removed T-Core disposition-merge proposed-final-comment-period Currently awaiting signoff of a majority of team members in order to enter the final comment period. labels Dec 12, 2018
@mscbot mscbot added the final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. label Dec 12, 2018
@mscbot
Copy link
Collaborator

mscbot commented Dec 12, 2018

🔔 This is now entering its final comment period, as per the review above. 🔔

@richvdh richvdh removed the final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. label Dec 12, 2018
@richvdh
Copy link
Member

richvdh commented Dec 12, 2018

no, mscbot.

@anoadragon453
Copy link
Member

I mean, it's just following orders.

@richvdh richvdh self-assigned this Dec 13, 2018
@mscbot
Copy link
Collaborator

mscbot commented Dec 17, 2018

The final comment period, with a disposition to merge, as per the review above, is now complete.

@richvdh
Copy link
Member

richvdh commented Dec 17, 2018

right, merging this as the FCP is complete.

@richvdh richvdh merged commit 3fda4a3 into master Dec 17, 2018
@afranke afranke deleted the rav/proposal/cs_api_in_login branch September 22, 2021 10:30
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.

6 participants