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

Optional email entry is considered mandatory #6969

Closed
ouchadam opened this issue Aug 30, 2022 · 0 comments · Fixed by #6981
Closed

Optional email entry is considered mandatory #6969

ouchadam opened this issue Aug 30, 2022 · 0 comments · Fixed by #6981
Labels
A-Onboarding O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-FTUE Issue is relevant to the first time use project or experience

Comments

@ouchadam
Copy link
Contributor

ouchadam commented Aug 30, 2022

Steps to reproduce

  1. With a homeserver with a mandatory=false email stage
  2. Attempt to create an account
  3. Notice the email address must be entered

2022-08-30T14:52:19,484388920+01:00

Synapse config to enable optional email registration step

with an email delegate

account_threepid_delegates:
    email: https://matrix.org

or email section

email:
  smtp_host: '############'
  smtp_port: 587
  smtp_user: '############'
  smtp_pass: '############'
  enable_tls: true
  require_transport_security: true
  notif_from: '############'
  app_name: '############'
  enable_notifs: false
  notif_for_new_users: false
  client_base_url: '############'
  validation_token_lifetime: 15m
  invite_client_location: 'https://matrix.org'

Outcome

What did you expect?

Allow the email to be optionally entered

What happened instead?

The email must be entered

Your phone model

No response

Operating system version

No response

Application version and app store

1.4.34

Homeserver

No response

Will you send logs?

No

Are you willing to provide a PR?

No

@ouchadam ouchadam added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-FTUE Issue is relevant to the first time use project or experience S-Minor Impairs non-critical functionality or suitable workarounds exist O-Uncommon Most users are unlikely to come across this or unexpected workflow A-Onboarding labels Aug 30, 2022
@RiotRobot RiotRobot added this to Incoming in Issue triage Aug 30, 2022
@ouchadam ouchadam moved this from Incoming to Triaged in Issue triage Aug 30, 2022
Issue triage automation moved this from Triaged to Closed Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Onboarding O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-FTUE Issue is relevant to the first time use project or experience
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant