checkout-js icon indicating copy to clipboard operation
checkout-js copied to clipboard

made taxes field customizable and fixed language error in de.json

Open pauliusinc opened this issue 2 years ago • 3 comments

What?

PR for customizable taxes field by language json files.

Why?

Closes: https://github.com/bigcommerce/checkout-js/issues/885

Testing / Proof

No need because few changes of strings

@bigcommerce/checkout

pauliusinc avatar May 28 '22 18:05 pauliusinc

Hello, we're looking into the wording adjustment. Meanwhile, to pass the unit test, you need to update snapshots with this command npm run test -- -u.

bc-peng avatar May 29 '22 23:05 bc-peng

Hey @pauliusinc is this still relevant?

animesh1987 avatar Sep 02 '22 00:09 animesh1987

Hey @animesh1987,

the language tag for the taxes is not relevant anymore. This will be set at another position (in the settings of the store).

However, the subtotal text is still relevant and is wrong translated to German within the checkout, the cart page, all marketing emails and for the printable invoice from the shop orders and also for the invoice that is downloadable for the customer on the shop itself.

I wrote many scripts to correct this wrong translation in all this positions. If this could be fixed I would really appreciate that.

pauliusinc avatar Sep 02 '22 06:09 pauliusinc

Thanks @pauliusinc for the feedback, seems like something that we need to work with our translation team since it seems like it's just not related to this repo but a wider issue.

I will close this PR in that case. And hopefully we can fix it at the root.

Again thanks for notifying us 🙇

animesh1987 avatar Sep 09 '22 08:09 animesh1987

Hi folks. We need to have an issue for this case.

funivan avatar Sep 09 '22 11:09 funivan

Thanks @funivan

animesh1987 avatar Sep 09 '22 11:09 animesh1987

@animesh1987 @funivan was this ticket created and rectified?

bc-0dp avatar Feb 13 '24 09:02 bc-0dp

@bc-0dp seems to have been fixed https://github.com/bigcommerce/checkout-js/blob/master/packages/locale/src/translations/de.json#L77 probably via the bot.

animesh1987 avatar Feb 13 '24 10:02 animesh1987

Hi @funivan I will close #885 for now. If this issue persists, then don't hesitate to create a new issue. ❤️

bc-0dp avatar Feb 13 '24 11:02 bc-0dp