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

Add internationalization to payment method clean name #6540

Conversation

PauloVilarinho
Copy link
Contributor

What? Why?

Closes #6535

This adds internationalization to the payment method model clean name method, fixing the internationalzation of the new payment method at the admin panel

What should we test?

Test create new payment method at admin panel

Release notes

Internationalization added at creating new payment method at admin panel for the payment method type

Changelog Category: User facing changes

Paulo Vilarinho added 2 commits December 17, 2020 00:34
The clean name payment method tests now uses the internationalization string
The clean name payment method now uses the internationalization string
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
@openfoodfoundation openfoodfoundation deleted a comment from hound bot Dec 17, 2020
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
spec/models/spree/payment_method_spec.rb Show resolved Hide resolved
Copy link
Contributor

@Matt-Yorkley Matt-Yorkley left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @PauloVilarinho ❤️

Copy link
Contributor

@andrewpbrett andrewpbrett left a comment

Choose a reason for hiding this comment

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

Thanks @PauloVilarinho !

@andrewpbrett
Copy link
Contributor

andrewpbrett commented Dec 18, 2020

@PauloVilarinho I moved this to the Test Ready column on our ZenHub. It may be a little while before it gets merged and deployed since not as many people are around in the next couple weeks, but it'll get in there soon! And we can take a look at the other PR you sent in the meantime. Thanks again! 🎉

@filipefurtad0 filipefurtad0 self-assigned this Dec 18, 2020
@filipefurtad0 filipefurtad0 added the pr-staged-uk staging.openfoodnetwork.org.uk label Dec 18, 2020
@filipefurtad0
Copy link
Contributor

Olá @PauloVilarinho ,
Welcome!

Thanks for fixing this. I see you added the keys for this in en.yml, the page renders correctly, so all should work once the translations are updated on Transifex:

image

The arrow above spots another missing translation - unrelated to this PR.

Looks good, moving to ready to go!
Thank you 🎉

@filipefurtad0 filipefurtad0 removed the pr-staged-uk staging.openfoodnetwork.org.uk label Dec 18, 2020
@sauloperez sauloperez merged commit 9a199ae into openfoodfoundation:master Dec 18, 2020
@PauloVilarinho PauloVilarinho deleted the add-internationalization-to-payment-method-clean-name branch December 18, 2020 22:16
@RachL
Copy link
Contributor

RachL commented Jan 14, 2021

@Matt-Yorkley @andrewpbrett I see this has been included in the release deployed on Tuesday yet I don't see the strings in Transifex. Can you confirm which strings I should be looking for?

@sigmundpetersen
Copy link
Contributor

I see the auto update of en.yml on Transifex has been turned off since Dec 9th.
I guess this is done while we're trying to find the best process for translation between releases.
But it means there has not been pushed any strings to Transifex since that date ⚠️

@andrewpbrett
Copy link
Contributor

Ohhh good catch @sigmundpetersen. I was just opening transifex to check the keys; see also this discussion: #6508 (comment)

@sauloperez
Copy link
Contributor

would you mind turning it on meanwhile @sigmundpetersen ?

@andrewpbrett
Copy link
Contributor

Is there an admin login to Transifex? I don't see the button to turn the updating back on, and I don't see an admin login in bitwarden. Or maybe someone could make me an admin in Transifex :)

@sigmundpetersen
Copy link
Contributor

I added back the url, will wait and see if it updates.
I added you as admin on Transifex @andrewpbrett , you should see the button now 🙂

@sigmundpetersen
Copy link
Contributor

the check happens 4 times a day

I guess we will have to wait a while 🙈
Or we can push an update with the command line client
https://docs.transifex.com/projects/updating-content/

@andrewpbrett
Copy link
Contributor

Okay I just ran tx push -s with the latest master and I see the string spree.admin.payment_methods.providers.stripesca, for example, in Transifex.

@RachL
Copy link
Contributor

RachL commented Jan 14, 2021

Along with 24 other new strings 😅 yay! Thanks guys

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.

Missing Translation: Payment Method types
7 participants