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
Change "see issue #5613" to "see details" #6657
Change "see issue #5613" to "see details" #6657
Conversation
…to-see-more-details
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.
LGTM!
<a | ||
className="inline-flex text-sm font-semibold text-black underline dark:text-white" | ||
href={`https://github.com/electricitymaps/electricitymaps-contrib/issues/${outageData.issue}`} | ||
> | ||
<span className="pl-1 underline">issue #{outageData.issue}</span> | ||
<span className="pl-1 underline">{t('estimation-card.outage-details')}</span> |
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.
Nice to get the whole string translatable as well!
Thanks for fixing it 👏 |
…to-see-more-details
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.
Looks good, might be nice to add a German translation here as they're probably the most affected by outages
Wouldn't it be weird if the message is in english and then it says see details in German? @tonypls |
Issue
AVO-171
Description
This PR changes the outage message so it says "see details" instead of "see issue #5613".
I thought "see details" was more correct than "see more details", but I can change it if you don't agree.
Preview
This picture is a mock since there are no outages at the moment (as far as I can tell).
Double check
pnpx prettier@2 --write .
andpoetry run format
in the top level directory to format my changes.