-
Notifications
You must be signed in to change notification settings - Fork 2
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
LINK-2104 | Prevent to delete signup after event start_time #168
Conversation
LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr168-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedregistrations-pr168-ui.dev.hel.ninja 😆🎉🎉🎉 |
ae0c257
to
d582a3d
Compare
LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr168-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedregistrations-pr168-ui.dev.hel.ninja 😆🎉🎉🎉 |
public/locales/en/signup.json
Outdated
"closedWithEnrolmentTime": "Registration for this event opens on {{openingDate}} at {{openingTime}}." | ||
"closedWithEnrolmentTime": "Registration for this event opens on {{openingDate}} at {{openingTime}}.", | ||
"eventStarted": "The event has already started and registration cannot be cancelled.", | ||
"hasPaymentCancellation": "The payment will be cancelled and cannot be modified.", |
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.
"hasPaymentCancellation": "The payment will be cancelled and cannot be modified.", | |
"hasPaymentCancellation": "The payment is being cancelled and cannot be modified.", |
Present tense could be used here like in the Finnish sentence.
public/locales/en/signup.json
Outdated
"closedWithEnrolmentTime": "Registration for this event opens on {{openingDate}} at {{openingTime}}.", | ||
"eventStarted": "The event has already started and registration cannot be cancelled.", | ||
"hasPaymentCancellation": "The payment will be cancelled and cannot be modified.", | ||
"hasPaymentRefund": "The payment will be refunded and cannot be modified.", |
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.
"hasPaymentRefund": "The payment will be refunded and cannot be modified.", | |
"hasPaymentRefund": "The payment is being refunded and cannot be modified.", |
public/locales/sv/signup.json
Outdated
"closedWithEnrolmentTime": "Anmälan till detta evenemang öppnar den {{openingDate}} kl {{openingTime}}." | ||
"closedWithEnrolmentTime": "Anmälan till detta evenemang öppnar den {{openingDate}} kl {{openingTime}}.", | ||
"eventStarted": "Evenemanget har redan börjat och registreringen kan inte avbrytas.", | ||
"hasPaymentCancellation": "Registreringsavgiften kommer att annulleras och kan inte ändras.", |
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.
"hasPaymentCancellation": "Registreringsavgiften kommer att annulleras och kan inte ändras.", | |
"hasPaymentCancellation": "Anmälningsavgiften annulleras och kan inte ändras.", |
Present tense form according to ChatGPT.
public/locales/sv/signup.json
Outdated
"closedWithEnrolmentTime": "Anmälan till detta evenemang öppnar den {{openingDate}} kl {{openingTime}}.", | ||
"eventStarted": "Evenemanget har redan börjat och registreringen kan inte avbrytas.", | ||
"hasPaymentCancellation": "Registreringsavgiften kommer att annulleras och kan inte ändras.", | ||
"hasPaymentRefund": "Anmälningsavgiften kommer att återbetalas och kan inte ändras.", |
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.
"hasPaymentRefund": "Anmälningsavgiften kommer att återbetalas och kan inte ändras.", | |
"hasPaymentRefund": "Anmälningsavgiften återbetalas och kan inte ändras.", |
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.
Some suggestions about translations, but overall looks good!
d582a3d
to
fc429e9
Compare
|
LINKEDREGISTRATIONS-UI branch is deployed to platta: https://linkedregistrations-pr168-ui.dev.hel.ninja 🚀🚀🚀 |
e2e tests result is success for https://linkedregistrations-pr168-ui.dev.hel.ninja 😆🎉🎉🎉 |
Description
Prevent to delete a signup or a signup group after an event has started.
Closes
LINK-2104