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

UR-2013 - Tweak Logout endpoint translation issues #800

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

suvedisamyog
Copy link
Contributor

All Submissions:

Changes proposed in this Pull Request:

Closes #
Previously when my-account endpoint was translated using WPML. Endpoints were not working and during logout when confirm and logout new popup was not showing and users was not logged out. This PR Fixes this issues for more
https://themegrill.atlassian.net/browse/UR-2013

How to test the changes in this Pull Request:

1.Install WPML multilanguage and string translation .(You can find zips in above jira tickets)
2.Translate user-registration plugin or translate only the endpoints.(Note: Translate user registration endpoints that are registered in WordPress endpoints like user-logout , profile-details or new endpoints saved on general settings endpoints.)
3.Make sure your page where my-account is translated and endpoints too.
4.Change language using flag switcher provided by wpml and verify all translated slugs are working properly (Note: Once endpoints are translated please save permalink once)

Types of changes:

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (modification of the currently available functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully ran tests with your changes locally?
  • Have you updated the documentation accordingly?

Changelog entry

Enter a short summary of all changes on this Pull Request. This will appear in the changelog if accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant