Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Config option to disable requesting MSISDN on registration #4423

Merged
merged 2 commits into from
Jan 22, 2019

Conversation

neilisfragile
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Jan 21, 2019

Codecov Report

Merging #4423 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #4423      +/-   ##
===========================================
+ Coverage    73.64%   73.65%   +0.01%     
===========================================
  Files          300      300              
  Lines        29801    29803       +2     
  Branches      4896     4896              
===========================================
+ Hits         21947    21952       +5     
+ Misses        6417     6411       -6     
- Partials      1437     1440       +3

@neilisfragile neilisfragile requested a review from a team January 21, 2019 15:30
Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm modulo my concerns re testing in #synapse-dev

@neilisfragile
Copy link
Contributor Author

tested with ios and android - all good. Can't test fully against riot web since it hard codes the msisdn field. The redesign will solve with this, and should modify the form based on the server's capabilities.

With that in mind I will merge but not enable on matrix.org

@neilisfragile neilisfragile merged commit 388c164 into develop Jan 22, 2019
@DMRobertson DMRobertson deleted the neilj/disable_msisdn_on_registration branch June 28, 2022 11:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants