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

Forgets homeserver url when creating account #8929

Closed
michaelkaye opened this issue Feb 25, 2019 · 4 comments
Closed

Forgets homeserver url when creating account #8929

michaelkaye opened this issue Feb 25, 2019 · 4 comments

Comments

@michaelkaye
Copy link
Contributor

Description

Custom server flows reset to matrix.org too often:

Steps to reproduce

  • Visit riot.im/app
  • Click sign in
  • Enter custom HS and IS urls
  • Realise you need to create an account, click "Create Account" text
  • Page has reset to "create account on matrix.org" - despite coming from custom HS/IS url page
  • Clicking Advanced - page has not remembered the custom HS/IS url here either

Describe how what happens differs from what you expected.

I would expect creating an account from the sign-in page to remember my HS and IS urls.

Doing the reverse - going from create an account back to the signing page does remember the HS and IS urls, which is good.

Version information

  • Platform: web (in-browser)

For the web app:

  • Browser: Firefox
  • OS: Ubuntu
  • URL: riot.im/app
@jryans
Copy link
Collaborator

jryans commented Feb 26, 2019

Thanks for the report. This sounds like a duplicate of #8695.

@jryans jryans closed this as completed Feb 26, 2019
@jryans
Copy link
Collaborator

jryans commented Feb 27, 2019

@michaelkaye I can no longer reproduce this on https://riot.im/develop (still happens on /app for now). Do you agree? (There are several reports of this, so verifying it's fixed would be great.)

@michaelkaye
Copy link
Contributor Author

I cannot replicate on /develop at the moment - the options default to the custom HS/IS and stay there until i click "matrix.org" which does then reset them (but that's probably OK)

@jryans
Copy link
Collaborator

jryans commented Feb 27, 2019

Thanks, sounds like it's been fixed by other changes then.

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

No branches or pull requests

2 participants