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

Testing #167

Open
wants to merge 10 commits into
base: master
from
Open

Testing #167

wants to merge 10 commits into from

Conversation

@Josue-T
Copy link
Collaborator

Josue-T commented Jan 2, 2020

Problem

  • Change-url was not supported because we can't change the server name. But since the domain is separated to the server name it's possible to change the domain.

Solution

  • Add change-url script
  • Fix some dirty workaround about the SSO

PR Status

  • Code finished.
  • Tested with Package_check.
  • Fix or enhancement tested.
  • Upgrade from last version tested.
  • Can be reviewed and tested.

Validation


Minor decision

  • Upgrade previous version :
  • Code review :
  • Approval (LGTM) :
  • Approval (LGTM) :
  • CI succeeded :
    Build Status

When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.

Josue-T added 9 commits Dec 14, 2019
We can't do this test because the path is fixed '/_matrix'.$
So the change-url script is only used to change the domain.
Change url
@Josue-T Josue-T added the pending label Jan 2, 2020
Upgrade synapse to 1.7.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.