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

1248 - Add locale to stripe return urls #1250

Conversation

dimitur2204
Copy link
Contributor

Closes #1248

Motivation and context

Added the language to the returnUrls of the stripe requests.

Testing

Tried both failed and success states and they are now translated to the language you had set when you come back.

Steps to test

Go through the donation flow to either succeed or fail.

Affected urls

http://localhost:3040/en/campaigns/donation/:campaignSlug

@github-actions
Copy link

github-actions bot commented Dec 12, 2022

✅ Tests will run for this PR. Once they succeed it can be merged.

@dimitur2204 dimitur2204 self-assigned this Dec 12, 2022
@dimitur2204 dimitur2204 added area: donations Regarding something that includes the donation flow run tests Allows running the tests workflows for forked repos labels Dec 12, 2022
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Dec 12, 2022
@dimitur2204 dimitur2204 changed the title 1248 - add locale to stripe return urls 1248 - Add locale to stripe return urls Dec 12, 2022
Copy link
Member

@kachar kachar left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution @dimitur2204

I was wondering if the default language is recognized when we provide it explicitly. I've tested that and it's all good. ⚙️

@kachar kachar merged commit 6d7f488 into podkrepi-bg:master Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: donations Regarding something that includes the donation flow
Projects
None yet
2 participants