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

MBS-10562: Add phrases for future (sidebar) dates #1348

Merged
merged 2 commits into from Aug 3, 2020

Conversation

reosarevok
Copy link
Member

MBS-10562

This only adds the option for cases that seem likely to happen and where we have custom phrases already. The opening and closing of a place can be announced ahead of time, and so can the dissolution of a group (but it would be strange to announce the foundation of a group before it happens, very dodgy to announce the death of a person and pretty much impossible to specify the birth of one).

@reosarevok reosarevok added the QoL Non-urgent quality of life improvements label Jan 12, 2020
@reosarevok reosarevok changed the title MBS-10562: Add phrases for future (sidebar) dates Implement MBS-10562: Add phrases for future (sidebar) dates Jan 12, 2020
@reosarevok reosarevok changed the title Implement MBS-10562: Add phrases for future (sidebar) dates MBS-10562: Add phrases for future (sidebar) dates Jun 1, 2020
Copy link
Contributor

@yvanzo yvanzo left a comment

Choose a reason for hiding this comment

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

Code looks good but added localizable messages are missing context group and place.
Please add context to the new localizable messages at least.
Feel free to change the preexisting messages that should have the same context if you want.

This only adds the option for cases that seem likely to happen and
where we have custom phrases already. The opening and closing of a
place can be announced ahead of time, and so can the dissolution
of a group (but it would be strange to announce the foundation
of a group before it happens, very dodgy to announce the death of
a person and pretty much impossible to specify the birth of one).
Copy link
Contributor

@yvanzo yvanzo left a comment

Choose a reason for hiding this comment

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

Thank you!

@reosarevok reosarevok merged commit 27e9acf into metabrainz:master Aug 3, 2020
@reosarevok reosarevok deleted the MBS-10562 branch August 3, 2020 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QoL Non-urgent quality of life improvements
Projects
None yet
3 participants