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

[PM-15180] - update copy for free trial banner #12151

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
class="tw-cursor-pointer"
rel="noreferrer noopener"
>
{{ "routeToPaymentMethodTrigger" | i18n }}
{{ "routeToPaymentMethodTriggerV2" | i18n }}
</a>
</bit-banner>
<app-header></app-header>
Expand Down
6 changes: 3 additions & 3 deletions apps/web/src/app/billing/services/trial-flow.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,11 +52,11 @@ export class TrialFlowService {

getFreeTrialMessage(trialRemainingDays: number): string {
if (trialRemainingDays >= 2) {
return this.i18nService.t("freeTrialEndPrompt", trialRemainingDays);
return this.i18nService.t("freeTrialEndPromptV2", trialRemainingDays);
} else if (trialRemainingDays === 1) {
return this.i18nService.t("freeTrialEndPromptForOneDayNoOrgName");
return this.i18nService.t("freeTrialEndPromptForOneDayNoOrgNameV2");
} else {
return this.i18nService.t("freeTrialEndingSoonWithoutOrgName");
return this.i18nService.t("freeTrialEndingSoonWithoutOrgNameV2");
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
class="tw-cursor-pointer"
rel="noreferrer noopener"
>
{{ "routeToPaymentMethodTrigger" | i18n }}
{{ "routeToPaymentMethodTriggerV2" | i18n }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Could not find this in messages.json

</a>
</bit-banner>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
rel="noreferrer noopener"
class="tw-cursor-pointer"
>
{{ "routeToPaymentMethodTrigger" | i18n }}
{{ "routeToPaymentMethodTriggerV2" | i18n }}
</a>
</bit-banner>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,14 +69,17 @@ export class VaultBannersComponent implements OnInit {
freeTrialMessage(organization: FreeTrial) {
if (organization.remainingDays >= 2) {
return this.i18nService.t(
"freeTrialEndPromptAboveTwoDays",
"freeTrialEndPromptAboveTwoDaysV2",
organization.organizationName,
organization.remainingDays.toString(),
);
} else if (organization.remainingDays === 1) {
return this.i18nService.t("freeTrialEndPromptForOneDay", organization.organizationName);
return this.i18nService.t("freeTrialEndPromptForOneDayV2", organization.organizationName);
} else {
return this.i18nService.t("freeTrialEndPromptForLessThanADay", organization.organizationName);
return this.i18nService.t(
"freeTrialEndPromptForLessThanADayV2",
organization.organizationName,
);
}
}

Expand Down
2 changes: 1 addition & 1 deletion apps/web/src/app/vault/org-vault/vault.component.html
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
class="tw-cursor-pointer"
rel="noreferrer noopener"
>
{{ "routeToPaymentMethodTrigger" | i18n }}
{{ "routeToPaymentMethodTriggerV2" | i18n }}
</a>
</bit-banner>
</ng-container>
Expand Down
82 changes: 52 additions & 30 deletions apps/web/src/locales/en/messages.json
Original file line number Diff line number Diff line change
Expand Up @@ -1642,27 +1642,9 @@
"passwordHistory": {
"message": "Password history"
},
"generatorHistory": {
"message": "Generator history"
},
"clearGeneratorHistoryTitle": {
"message": "Clear generator history"
},
"cleargGeneratorHistoryDescription": {
"message": "If you continue, all entries will be permanently deleted from generator's history. Are you sure you want to continue?"
},
"noPasswordsInList": {
"message": "There are no passwords to list."
},
"clearHistory": {
"message": "Clear history"
},
"nothingToShow": {
"message": "Nothing to show"
},
"nothingGeneratedRecently": {
"message": "You haven't generated anything recently"
},
"clear": {
"message": "Clear",
"description": "To clear something out. Example: To clear browser history."
Expand Down Expand Up @@ -3879,6 +3861,15 @@
}
}
},
"freeTrialEndPromptV2": {
"message": "Your free trial ends in $COUNT$ days.",
"placeholders": {
"count": {
"content": "$1",
"example": "You must set up 2FA on your user account before you can join this organization."
Copy link
Contributor

Choose a reason for hiding this comment

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

i just noticed, this example text is wrong.

}
}
},
"freeTrialEndPromptAboveTwoDays": {
"message": "$ORGANIZATION$, your free trial ends in $COUNT$ days. To maintain your subscription,",
Copy link
Contributor

@cyprain-okeke cyprain-okeke Nov 25, 2024

Choose a reason for hiding this comment

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

Did business requested that To maintain your subscription, be removed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@cyprain-okeke Given how different languages break up sentences in different ways and the fact this translation in particular was broken between 2 keys it was decided to make both keys complete sentences in order to allow for proper translation.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please provide a video recording

Copy link
Collaborator Author

@jaasen-livefront jaasen-livefront Nov 25, 2024

Choose a reason for hiding this comment

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

@cyprain-okeke Sorry I'm not sure what you mean. What would you like the recording of? I can provide a screenshot of what new copy looks like if that's what you'd like.

Copy link
Contributor

Choose a reason for hiding this comment

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

run the application and test it, while testing record it and attached it to the pr.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@cyprain-okeke Screen recording attached

"placeholders": {
Expand All @@ -3892,6 +3883,19 @@
}
}
},
"freeTrialEndPromptAboveTwoDaysV2": {
"message": "$ORGANIZATION$, your free trial ends in $COUNT$ days.",
"placeholders": {
"count": {
Copy link
Contributor

Choose a reason for hiding this comment

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

the example text is wrong for the two placeholders.

"content": "$2",
"example": "organization name"
},
"organization": {
"content": "$1",
"example": "remaining days"
}
}
},
"freeTrialEndPromptForOneDay": {
"message": "$ORGANIZATION$, your free trial ends tomorrow. To maintain your subscription,",
"placeholders": {
Expand All @@ -3901,9 +3905,21 @@
}
}
},
"freeTrialEndPromptForOneDayV2": {
"message": "$ORGANIZATION$, your free trial ends tomorrow.",
"placeholders": {
"organization": {
"content": "$1",
"example": "organization name"
}
}
},
"freeTrialEndPromptForOneDayNoOrgName": {
"message": "Your free trial ends tomorrow. To maintain your subscription,"
},
"freeTrialEndPromptForOneDayNoOrgNameV2": {
"message": "Your free trial ends tomorrow."
},
"freeTrialEndPromptForLessThanADay": {
"message": "$ORGANIZATION$, your free trial ends today. To maintain your subscription,",
"placeholders": {
Expand All @@ -3913,12 +3929,27 @@
}
}
},
"freeTrialEndPromptForLessThanADayV2": {
"message": "$ORGANIZATION$, your free trial ends today.",
"placeholders": {
"organization": {
"content": "$1",
"example": "organization name"
}
}
},
"freeTrialEndingSoonWithoutOrgName": {
"message": "Your free trial ends today. To maintain your subscription,"
},
"freeTrialEndingSoonWithoutOrgNameV2": {
"message": "Your free trial ends today."
},
"routeToPaymentMethodTrigger": {
"message": "add a payment method."
},
"routeToPaymentMethodTriggerV2": {
"message": "Click here add a payment method."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"message": "Click here add a payment method."
"message": "Click here to add a payment method."

},
"joinOrganization": {
"message": "Join organization"
},
Expand Down Expand Up @@ -8069,18 +8100,9 @@
"loginInitiated": {
"message": "Login initiated"
},
"rememberThisDeviceToMakeFutureLoginsSeamless": {
"message": "Remember this device to make future logins seamless"
},
"deviceApprovalRequired": {
"message": "Device approval required. Select an approval option below:"
},
"deviceApprovalRequiredV2": {
"message": "Device approval required"
},
"selectAnApprovalOptionBelow": {
"message": "Select an approval option below"
},
"rememberThisDevice": {
"message": "Remember this device"
},
Expand Down Expand Up @@ -8310,9 +8332,6 @@
"userEmailMissing": {
"message": "User email missing"
},
"activeUserEmailNotFoundLoggingYouOut": {
"message": "Active user email not found. Logging you out."
},
"deviceTrusted": {
"message": "Device trusted"
},
Expand Down Expand Up @@ -8408,6 +8427,9 @@
"collectionManagementDesc": {
"message": "Manage the collection behavior for the organization"
},
"limitCollectionCreationDeletionDesc": {
"message": "Limit collection creation and deletion to owners and admins"
},
"limitCollectionCreationDesc": {
"message": "Limit collection creation to owners and admins"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
(click)="navigateToPaymentMethod()"
rel="noreferrer noopener"
>
{{ "routeToPaymentMethodTrigger" | i18n }}
{{ "routeToPaymentMethodTriggerV2" | i18n }}
</a>
</bit-banner>
</ng-container>
Expand Down
Loading