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

fix: password validation, add autofocus #117

Merged
merged 2 commits into from
Oct 29, 2021
Merged

fix: password validation, add autofocus #117

merged 2 commits into from
Oct 29, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Oct 29, 2021

Clean up a little and fix when the Change Password button is enabled.
Add autofocus to TextBox component and add to password modal's first input.

Tossing in an email fallback when member name is unavailable.

@netlify
Copy link

netlify bot commented Oct 29, 2021

✔️ Deploy Preview for reearth-web ready!

🔨 Explore the source changes: 1769cc6

🔍 Inspect the deploy log: https://app.netlify.com/sites/reearth-web/deploys/617b6fb6ee5d0d000722d1da

😎 Browse the preview: https://deploy-preview-117--reearth-web.netlify.app/

rot1024
rot1024 previously approved these changes Oct 29, 2021
@codecov
Copy link

codecov bot commented Oct 29, 2021

Codecov Report

Merging #117 (1769cc6) into main (5d57c4f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage   44.81%   44.81%           
=======================================
  Files          56       56           
  Lines        1100     1100           
  Branches      174      174           
=======================================
  Hits          493      493           
  Misses        554      554           
  Partials       53       53           

@KaWaite KaWaite merged commit 3484540 into main Oct 29, 2021
@KaWaite KaWaite deleted the fix/password-val2 branch October 29, 2021 05:11
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
Co-authored-by: Renovate Bot <[email protected]>
Co-authored-by: yk <[email protected]>
Co-authored-by: rot1024 <[email protected]>
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