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.

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

Update error messages on user-facing forms #146

Merged
merged 7 commits into from
Feb 2, 2024

Conversation

WillNilges
Copy link
Collaborator

No description provided.

@WillNilges WillNilges changed the title [draft] Add/update response messages Update error messages on user-facing forms Feb 2, 2024
Copy link
Member

@Andrew-Dickinson Andrew-Dickinson left a comment

Choose a reason for hiding this comment

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

Huge +1 for the actual JSON objects instead of the weird psuedo-string things

@WillNilges
Copy link
Collaborator Author

yea haha oops

@WillNilges WillNilges merged commit a678f50 into main Feb 2, 2024
4 checks passed
WillNilges added a commit that referenced this pull request Feb 2, 2024
* Add/update response messages

* black

* fix tests and pelias

* white

* Update formatting of NN Assign responses

* s/message/detail/g

* couple more msgs
WillNilges added a commit that referenced this pull request Feb 2, 2024
* Stub out webpages file

* Create meshweb to serve pages and manage content

* white

* Bootstrap?

I really need to go to bed

* Update error messages on user-facing forms (#146)

* Add/update response messages

* black

* fix tests and pelias

* white

* Update formatting of NN Assign responses

* s/message/detail/g

* couple more msgs

* Add logo, update landing page

* Fix static file serving in development

* Build static files into docker image

* Add margin
@WillNilges WillNilges deleted the willnilges/join-form-msgs branch February 19, 2024 02:03
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.

2 participants