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

[IMPROVE] Status Text form validation #16121

Merged
merged 4 commits into from
Jan 8, 2020

Conversation

gabriellsh
Copy link
Member

@gabriellsh gabriellsh commented Jan 2, 2020

Status Text was fixed added missing fields to the verification.

Closes #16057

Now it's only possible to change password if both fields match. Additionally, it was possible to save even if no field was changed, that was fixed and added missing fields to the verification.

MartinSchoeler
MartinSchoeler previously approved these changes Jan 2, 2020
Copy link
Contributor

@MartinSchoeler MartinSchoeler left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@rodrigok rodrigok changed the title [FIX]Save password without confirmation [FIX] Save password without confirmation Jan 7, 2020
@ashwaniYDV
Copy link
Contributor

ashwaniYDV commented Jan 7, 2020

@gabriellsh I have already made the Pull Request for the issue created by me. 😀 It's pending for a long time. @rodrigok Would you please review it.
PR #16060

@ggazzo ggazzo changed the title [FIX] Save password without confirmation [IMPROVE] Status Text form validation Jan 8, 2020
@ggazzo ggazzo added this to the 3.0.0 milestone Jan 8, 2020
@ggazzo ggazzo merged commit db0862e into develop Jan 8, 2020
@ggazzo ggazzo deleted the fix/save_password_without_confirm branch January 8, 2020 11:38
ggazzo pushed a commit that referenced this pull request Jan 8, 2020
gabriellsh added a commit that referenced this pull request Jan 9, 2020
…default-favorite-channels

* 'develop' of github.com:RocketChat/Rocket.Chat: (31 commits)
  fix space
  update on mongo, node and caddy
  [NEW] Enforce plain text emails converting from HTML when no text version supplied (#16063)
  [IMPROVE] Status Text form validation (#16121)
  [FIX] auto translate cache (#15768)
  [FIX] Save new password without confirmation (#16060)
  Disable PR Docker image build (#16141)
  [IMPROVE] Set the color of the cancel button on modals to #bdbebf for enhanced  visibiity (#15913)
  [FIX] Break message-attachment text to the next line (#16039)
  [IMPROVE] Remove NRR (#16071)
  Add Cloud Info to translation dictionary (#16122)
  [FIX] Resolving console errors Administrations (#16139)
  [FIX] SafePorts: Ports 80, 8080 & 443 linked to respective protocols (#16108)
  [FIX] Drag and drop disabled when file upload is disabled (#16049)
  [FIX] Video message sent to wrong room (#16113)
  [FIX] "User not found" for direct messages (#16047)
  [BREAK] Removed room counter from sidebar (#16036)
  [FIX] Embedded style when using 'go' command  (#16051)
  [NEW]  Setting Top navbar in embedded mode (#16064)
  [FIX] Thread message icon overlapping text (#16083)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 15, 2020
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.

Password is updated even if the password and confirm password are not equal
4 participants