Skip to content

Conversation

@chuckbutkus
Copy link
Collaborator

@chuckbutkus chuckbutkus commented Jun 6, 2025

This PR adds email validation to the user settings page:

  • Added client-side email validation using regex pattern
  • Added visual feedback for invalid email formats
  • Disabled the save button when the email format is invalid
  • Added translations for the validation error message

These changes improve the user experience by providing immediate feedback on email format validity and prevent submission of invalid email addresses.


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:539d620-nikolaik   --name openhands-app-539d620   docker.all-hands.dev/all-hands-ai/openhands:539d620

chuckbutkus and others added 30 commits May 30, 2025 15:43
…ied and show warning message when not verified
@openhands-ai
Copy link

openhands-ai bot commented Jun 6, 2025

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Lint

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #8952

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

@github-actions
Copy link
Contributor

This PR is stale because it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale Inactive for 40 days label Jul 11, 2025
@github-actions
Copy link
Contributor

This PR was closed because it has been stalled for over 30 days with no activity.

@github-actions github-actions bot closed this Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stale Inactive for 40 days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants