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

PC-I182: Change Text When Upgrading Account From 'Account Created' to 'Account Upgraded' #512

Merged
merged 2 commits into from
Nov 1, 2022

Conversation

owenlejeune
Copy link
Contributor

Description

Change text when upgrading from free to premium from 'Account Created' to 'Account Upgraded'

Fixes #182

Testing Instructions

  1. Upgrade a free account to premium

Screenshots or Screencast

Screenshot_20221029-084941

@owenlejeune owenlejeune requested a review from a team as a code owner October 29, 2022 12:53
Copy link
Contributor

@mchowning mchowning left a comment

Choose a reason for hiding this comment

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

This looks good and works well. Thanks for the contribution @owenlejeune ! 🙇

The only question I have is about where the translations came from. We use glotpress to get human-translated versions of our strings imported into Pocket Casts. Did you get these translations from something like Google Translate and add them manually? If so, I really appreciate you going to that effort, but let's remove all but the English version of the String from this PR and let GlotPress handle the translations for us.

@owenlejeune
Copy link
Contributor Author

@mchowning My bad! I wasn't sure what the process for adding translations was so I added them in myself, but I've removed them now and just let the entry in the default strings file.

@mchowning
Copy link
Contributor

My bad! I wasn't sure what the process for adding translations was so I added them in myself

No worries, we need to document this better. Thanks for the update.

@mchowning mchowning enabled auto-merge November 1, 2022 15:54
@mchowning mchowning merged commit 26d77bf into Automattic:main Nov 1, 2022
@owenlejeune owenlejeune deleted the i-182 branch November 1, 2022 16:24
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.

After upgrading an account to Plus, the confirmation screen says "Account Created"
2 participants