-
Notifications
You must be signed in to change notification settings - Fork 754
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 Localization Issues #4265
Fix Localization Issues #4265
Conversation
868cb58
to
c2cb40c
Compare
I've rebased these changes on the release branch |
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.
Looks good to me, but will definitely need some good testing
Strange the pipeline failed because of Dnn.ClientDependency, I gonna try to run it again to see if that is a fluke... |
/azp run |
No pipelines are associated with this pull request. |
Looks like it's failing because MyGet is down... |
Looks like MyGet is back up, firing up a new build |
/azp run |
No pipelines are associated with this pull request. |
I updated my local repo with your changes but now we have 1000+ changes ... Maybe you'll need to rebase again. |
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.
Looks like this was rebased on develop instead of the release branch... I will try to fix that if I can...
da5d855
to
2be936a
Compare
2be936a
to
a1c5c15
Compare
I've cherry-picked from your merge commit, @donker, and then rebased again on the release branch |
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.
Awesome, I downloaded this CI build and ran a few tests, looks fine to me in the context of a visitor, a regular user, an admin and a host both for WebAPI and WebForms.
Thanks guys.
During testing we discovered that the thread locale was not set correctly when switching languages. This PR addresses that issue.