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

add help us improve field to all registrations #1737

Merged

Conversation

kdachev97
Copy link
Contributor

Added a new tick box in Registration Forms for both EN and BG.

Before:
Screenshot 2024-03-13 at 10 51 05
Screenshot 2024-03-13 at 10 50 58

After:
Screenshot 2024-03-13 at 10 49 01
Screenshot 2024-03-13 at 10 49 57

Copy link

❌ Not all tests have run for this PR. Please add the run tests label to trigger them.

Copy link
Member

@sashko9807 sashko9807 left a 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:

If you have any further questions, regarding the requested changes feel free to contact me anytime.

Thank you for your contribution!

"eik-invalid": "Невалидно ЕИК"
}
"eik-invalid": "Невалидно ЕИК",
"help-us-improve": "Искате ли да ни помогнете да подобрим платформата? Искаме да чуем вашето мнение? Отделете ни 30 мин за подобрите платформа на подкрепи.бг"
Copy link
Member

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

Copy link
Member

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 минути, за подобрите платформа на Подкрепи.бг""

Copy link

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?"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@wqweto Този текст е вече променен в друг тикет: dbfc148

label={
<Typography variant="body2">
{t('validation:help-us-improve')}{' '}
{/* <ExternalLink href={routes.helpUsImprove}>
Copy link
Member

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.

@ani-kalpachka ani-kalpachka linked an issue Mar 17, 2024 that may be closed by this pull request
@sashko9807 sashko9807 added the run tests Allows running the tests workflows for forked repos label Mar 18, 2024
@sashko9807 sashko9807 added run tests Allows running the tests workflows for forked repos and removed run tests Allows running the tests workflows for forked repos labels Mar 18, 2024
Copy link
Member

@sashko9807 sashko9807 left a 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?

@sashko9807 sashko9807 added the run tests Allows running the tests workflows for forked repos label Mar 18, 2024
@sashko9807 sashko9807 merged commit b85caf3 into podkrepi-bg:master Mar 19, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: frontend run tests Allows running the tests workflows for forked repos
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a tick box and text to new registration
4 participants