Skip to content

fix: Updated eye icon#102

Merged
yp969803 merged 3 commits intofordevio:mainfrom
yashisrani:fix/passwordshower
Jul 22, 2025
Merged

fix: Updated eye icon#102
yp969803 merged 3 commits intofordevio:mainfrom
yashisrani:fix/passwordshower

Conversation

@yashisrani
Copy link
Copy Markdown
Contributor

Description

Fixes: #92

Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.

Type of change

What type of changes does your code introduce to the wharf? Put an x in the box that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Verification

Please check the testing guidelines for recommendations about automated tests.

@yashisrani yashisrani requested a review from yp969803 as a code owner July 9, 2025 18:44
@yp969803
Copy link
Copy Markdown
Collaborator

yp969803 commented Jul 9, 2025

please run "make format" and "make generate"

<div className="inputDiv" tabIndex={0}>
<input
type="password"
type={showPassword ? 'text' : 'password'}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
type={showPassword ? 'text' : 'password'}
type={showPassword ? '' : 'password'}

Copy link
Copy Markdown
Contributor Author

@yashisrani yashisrani Jul 12, 2025

Choose a reason for hiding this comment

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

@yp969803 bro, i staged 3 files.. or i add all changed files to commit ..

Screenshot 2025-07-12 at 11 41 29 PM

@yashisrani yashisrani requested a review from yp969803 July 12, 2025 18:23
@yp969803
Copy link
Copy Markdown
Collaborator

stage everything

@yp969803
Copy link
Copy Markdown
Collaborator

useless files are ignored by gitignore

@yashisrani
Copy link
Copy Markdown
Contributor Author

useless files are ignored by gitignore

i staged all thing .. now it's giving build errors ..

@yp969803
Copy link
Copy Markdown
Collaborator

ping @yashisrani

@yashisrani yashisrani force-pushed the fix/passwordshower branch from 2ebd71a to 54cee0d Compare July 21, 2025 18:09
@yp969803
Copy link
Copy Markdown
Collaborator

yp969803 commented Jul 21, 2025

run these commands

git rebase upstream/main

# solve merge conficts

git add .

git rebase --continue

git push -f origin fix/passwordshower

@yp969803
Copy link
Copy Markdown
Collaborator

ping @yashisrani

@yp969803
Copy link
Copy Markdown
Collaborator

yp969803 commented Jul 22, 2025

run make generate && make format

@yashisrani yashisrani force-pushed the fix/passwordshower branch from d926559 to 635bf21 Compare July 22, 2025 10:17
@yp969803 yp969803 force-pushed the fix/passwordshower branch from 529df32 to 971942a Compare July 22, 2025 10:35
@yp969803 yp969803 merged commit 6887f37 into fordevio:main Jul 22, 2025
7 checks passed
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.

Click on eye icon to show the password in login page

2 participants