-
Notifications
You must be signed in to change notification settings - Fork 93
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
add help us improve field to all registrations #1737
add help us improve field to all registrations #1737
Conversation
❌ Not all tests have run for this PR. Please add the |
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.
Well done. There are minor last-minute changes to the text, mentioned below, please look at them.
This commit should also be supplemented, by changes in the backend as well, in order to keep track of who actually has agreed to help, so if you can please do.
Needed changes to the backend are:
-
Adding helpUsImprove field, to the prisma model
-
Adding helpUsImprove property, to the registerDto
If you have any further questions, regarding the requested changes feel free to contact me anytime.
Thank you for your contribution!
public/locales/bg/validation.json
Outdated
"eik-invalid": "Невалидно ЕИК" | ||
} | ||
"eik-invalid": "Невалидно ЕИК", | ||
"help-us-improve": "Искате ли да ни помогнете да подобрим платформата? Искаме да чуем вашето мнение? Отделете ни 30 мин за подобрите платформа на подкрепи.бг" |
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.
It was discussed with the design team, it is better to replace 30 мин
with 30 минути
.
Same goes for the EN translation, 30 mins to
30 minutes
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.
I would update it to:
"Искате ли да ни помогнете да подобрим платформата? Искаме да чуем Вашето мнение? Отделете ни 30 минути, за подобрите платформа на Подкрепи.бг""
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.
"Искаме да чуем вашето мнение" and "We want to hear from you" are plain statements and don't need question mark unless the original intend was to read "Искате да чуем вашето мнение?" and "You want us to hear from you?"
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.
label={ | ||
<Typography variant="body2"> | ||
{t('validation:help-us-improve')}{' '} | ||
{/* <ExternalLink href={routes.helpUsImprove}> |
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.
Please remove the commented line.
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.
The automatic linter appears, to have failed. Do you mind running yarn lint
(on the frontend repo), and commit the changes?
Added a new tick box in Registration Forms for both EN and BG.
Before:
After: