Skip to content

Swathi lb dashboard login features#5063

Merged
one-community merged 14 commits intodevelopmentfrom
Swathi_LBDashboard_LoginFeatures
Apr 17, 2026
Merged

Swathi lb dashboard login features#5063
one-community merged 14 commits intodevelopmentfrom
Swathi_LBDashboard_LoginFeatures

Conversation

@SwathiAngadi
Copy link
Copy Markdown
Contributor

@SwathiAngadi SwathiAngadi commented Mar 26, 2026

Description

Enhance Login UX & Validation for Listing & Bidding Portal. With below features

  • Add client-side validation for email format and empty fields.
  • Implement show/hide password toggle.
  • Display inline error messages for failed login attempts.
  • Add loading/disabled state on Login button during authentication.
  • Show Caps Lock warning on password field.
  • Ensure keyboard accessibility and basic ARIA labels.

Related PRS (if any):

No related PRs

Main changes explained:

  1. Added live validation of input field on the first key stroke.
  2. Used validator library for email validation

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. Hit this URL: http://localhost:5173/lbdashboard/login
  5. Verify for different error scenarios.
  6. Verify user will be able to move with tabs
  7. Verify this page has implemented accessbility.
  8. Verify multiple calls are not made to backend while waiting for authentication.
    11.Check whether dark mode is implemented for this page.
  9. Test for Responsiveness of the page

Screenshots or videos of changes:

Screenshot 2026-03-25 122747 Screenshot 2026-03-25 122808 Screenshot 2026-03-25 122821 Screenshot 2026-03-25 122854 Screenshot 2026-03-25 122943 Screenshot 2026-03-25 122958 chrome_V9ntNuIq36
Screen.Recording.2026-03-25.123948.mp4
Screen.Recording.2026-03-25.123845.mp4

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 26, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit adb3eca
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69df3b52d679ac0008cba949
😎 Deploy Preview https://deploy-preview-5063--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 9, 2026
@rajanidi1999
Copy link
Copy Markdown

Hi Swathi,
The Login page for the Listing and Bidding Portal is displaying correctly with a clean and user-friendly layout. The email and password input fields are clearly visible and properly aligned, and the login button is easily accessible. The note about using Production/Main credentials is clearly mentioned, which helps guide users.
image

Anusha-Gali
Anusha-Gali previously approved these changes Apr 11, 2026
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Swathi,

I have reviewed the PR locally and it works as per requirement.

Image Image Image Image Image Image Image Image

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Swathi,

I tested this PR locally and everything is working as expected. Error scenarios, tab navigation, accessibility, dark mode, and responsiveness are all functioning correctly.

Image Image Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 02a3f74 into development Apr 17, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants