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

fix: Correct "processing changes" translation name #333

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

Manizuca
Copy link
Contributor

@Manizuca Manizuca commented Nov 9, 2020

Impact: minor
Type: *bugfix

Issue

While saving changes in some forms, "app.settings.saveProcessing" is visible for some milliseconds.

Solution

Change "app.settings.saveProcessing" to "admin.settings.saveProcessing", the correct name used in https://github.com/reactioncommerce/api-plugin-translations/blob/ef7ce859a1320b3effbd481de9dc51f188282f88/src/i18n/en.json#L57

@nnnnat
Copy link
Contributor

nnnnat commented Nov 9, 2020

@Manizuca thank you for the pull request. Can you please sign your commit to comply with our DCO?
https://github.com/reactioncommerce/reaction-admin/pull/333/checks?check_run_id=1374471198

@Manizuca
Copy link
Contributor Author

Manizuca commented Nov 9, 2020

oops, i forgot to signoff. Updated

@willopez willopez self-requested a review November 9, 2020 21:19
Copy link
Member

@willopez willopez 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 @Manizuca!

@willopez willopez merged commit a70b34e into reactioncommerce:trunk Nov 9, 2020
@mikemurray mikemurray mentioned this pull request Dec 4, 2020
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.

3 participants