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.

Sign up for GitHub

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

Jump to bottom

Show required field text and red outline after user attempt #1267

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Show required field text and red outline after user attempt #1267

wants to merge 4 commits into from

Conversation

gitcliff
Copy link
Contributor

@gitcliff gitcliff commented Sep 18, 2024 •
edited
Loading

Pull Requests Requirements

  • The PR title includes a brief description of the work done, including the
    Issue number if applicable.
  • The PR includes a video showing the changes for the work done.
  • The PR title follows conventional commit label standards.
  • The changes confirm to the OpenElis Global x3 Styleguide and design
    documentation.
  • The changes include tests or are validated by existing tests.
  • I have read and agree to the Contributing Guidelines of this project.

Summary

Screenshots

https://www.loom.com/share/f6c2175ebc1d411190ef63fbf2121bbd

Related Issue

#839

Other

[Add any additional information or notes here]

@gitcliff gitcliff mentioned this pull request Sep 19, 2024
Open
@mozzy11
Copy link
Collaborator

mozzy11 commented Oct 30, 2024

Thanks @gitcliff .
i think the vid is not accesible

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gitcliff @mozzy11