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

Change State to State/Province #39223

Closed
wants to merge 12 commits into from
Closed

Change State to State/Province #39223

wants to merge 12 commits into from

Conversation

mahsem
Copy link
Contributor

@mahsem mahsem commented Jan 8, 2024

Change State to State/Province and sync with frappe existing string

@github-actions github-actions bot added the needs-tests This PR needs automated unit-tests. label Jan 8, 2024
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Rate label to existing Tax Rate label so it can be correctly translated in other languages
Change Personal to Personal Details to correspond with fieldname  as Personal is to generic
@@ -253,4 +253,4 @@
"sort_order": "ASC",
"states": [],
"track_changes": 1
Copy link
Collaborator

Choose a reason for hiding this comment

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

Modified date has not updated, due to which this changes won't be reflect on the new site after the migrate

@deepeshgarg007
Copy link
Member

@mahsem Please make the change suggested by @rohitwaghchaure and then reopen the PR. Also use semantic commits.
https://github.com/frappe/erpnext/wiki/Pull-Request-Checklist

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs-tests This PR needs automated unit-tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants