-
Notifications
You must be signed in to change notification settings - Fork 89
Report Wizard #511
base: master
Are you sure you want to change the base?
Report Wizard #511
Conversation
Guys, this is insanely good! Blown away. Let's get this in! |
a11y: Can go through the form with just keyboard: awesome! Nits:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work, will be awesome to get this out 😄 Left some comments above ☝️
If the texts are ok, should we start gathering translations? |
@fossecode maybe we should wait for #537 to be merged before asking translations? |
Yes, I just have to add two more countries, look over it again and then it should be ready to merge. Will do it later tonight or tomorrow morning. |
I think we are ready to start gathering translations :) |
NL Dutch 🇳🇱
|
Norwegian 🇳🇴 (feels like the voting on Eurovision lol)
|
Remember to merge in the latest changes from the upstream before adding the translations, so that you get all the latest locale files. |
Just tested it out locally, feels very smooooth, much better than before 🙌💯🥇 |
en-US 🇺🇸
|
remove mb-0 class Co-Authored-By: Eirik Fosse <eirik@bustbyte.no>
Greek translations
Also typo in |
🇺🇦 version
|
this serves for all
|
🇵🇭 This is for
|
Looks awesome! However I cannot proceed past the first step with toast/notification "Please enter a valid postal code for your country" with a non-norwegian postal code. Also tried to reset the form. Probably related, in the console I receive an "Uncaught type error", related to I'm probably missing something here? |
🇩🇪
"Are you from Norway?" probably needs localization as well? |
This is fixed in the commit just above here. The problem was that this fork had become outdated in it's config schema. After updating the us config is correct which was the cause of the issue (old one was incorrectly set to accept only 4 digit zip codes when the us uses 5 digit ones). The error related to Thank you all for the translations we are getting much closer! |
LGTM now. |
Do we have an overview of which translations we are missing now? |
We have:
we need
I think this is a complete list (we can use existing en and es ones for all of the locales that use those languages |
some small improvements for being kind 🇩🇪
|
Many thanks to @zacel who did the majority of these changes, this PR proposes using a breadcrumb wizard for the report/form.
New breadcrumb/steps at the top for each page
User's progress through form is updated as they go, and they can click on successfully completed steps to go back
Required fields indicated with a *
Toast notifications (bottom right corner) for validation errors
Updated submit button
New strings added: