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

Fix home memo cards language update #232

Merged
merged 4 commits into from
Jul 14, 2022

Conversation

lanchana
Copy link
Contributor

No description provided.

@lanchana lanchana requested a review from cnguyen812 July 14, 2022 14:39
@lanchana lanchana force-pushed the fix-home-card-language-update branch 2 times, most recently from d5d0de4 to e6f1eeb Compare July 14, 2022 18:20
@lanchana lanchana force-pushed the fix-home-card-language-update branch from e6f1eeb to e1f9a22 Compare July 14, 2022 18:22
@@ -184,7 +184,7 @@ const Referral = ({}) => {

const getStatus = (status: string, expiration: number) => {
return status === 'pending' && currentDate >= expiration
? 'Expired'
? t('Expired')
: status.charAt(0).toUpperCase() + status.slice(1);
Copy link
Contributor

Choose a reason for hiding this comment

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

Is status enumerated somewhere? If so we should create a map and map each known status to a translated value

@@ -5,38 +5,47 @@ interface ItemProps {

export const LanguageList: Array<ItemProps> = [
{
// t('Dutch')
Copy link
Contributor

Choose a reason for hiding this comment

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

Why no translate?

@JohnathanWhite JohnathanWhite merged commit 0b3821a into bitpay:master Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants