-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Initial i18n for Fulfillment #6545
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
Conversation
Signed-off-by: Sujith <mail.sujithvn@gmail.com>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think one copypasta mistake
packages/api-plugin-fulfillment-method-pickup-store/src/i18n/en.json
Outdated
Show resolved
Hide resolved
@@ -245,7 +245,7 @@ importers: | |||
'@reactioncommerce/logger': link:../../packages/logger | |||
'@reactioncommerce/nodemailer': 5.0.5 | |||
'@reactioncommerce/random': link:../../packages/random | |||
'@snyk/protect': 1.1017.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably best to just remove this
Signed-off-by: Sujith <mail.sujithvn@gmail.com>
This PR is replaced by the commit |
Signed-off-by: Sujith mail.sujithvn@gmail.com
Resolves #6473
Impact: minor
Type: feature
Issue
There are no localised versions of customer or admin facing messages.
Solution
Created i18n config files for the newly introduced plugins.
The required files for two of the below mentioned plugins were completely copied over from old plugin.
api-plugin-fulfillment-type-shipping
(copied over fromapi-plugin-shipments
)api-plugin-fulfillment-method-shipping-flat-rate
(copied over fromapi-plugin-shipments-flat-rate
)Added new config files for remaining 4 plugins below
api-plugin-fulfillment
api-plugin-fulfillment-type-pickup
api-plugin-fulfillment-method-pickup-store
api-plugin-fulfillment-method-shipping-ups
Breaking changes
None
Testing
Need to co-ordinate with the Front-end team to understand if there are any additional entries required in the config-files for the new plugins.