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 Localization Issues #4265

Merged
merged 4 commits into from
Oct 31, 2020

Conversation

donker
Copy link
Contributor

@donker donker commented Oct 30, 2020

During testing we discovered that the thread locale was not set correctly when switching languages. This PR addresses that issue.

@donker donker added this to the 9.8.0 milestone Oct 30, 2020
@bdukes
Copy link
Contributor

bdukes commented Oct 30, 2020

I've rebased these changes on the release branch

Copy link
Contributor

@bdukes bdukes left a 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

@valadas
Copy link
Contributor

valadas commented Oct 30, 2020

Strange the pipeline failed because of Dnn.ClientDependency, I gonna try to run it again to see if that is a fluke...

@valadas
Copy link
Contributor

valadas commented Oct 30, 2020

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@valadas
Copy link
Contributor

valadas commented Oct 30, 2020

Looks like it's failing because MyGet is down...

@valadas
Copy link
Contributor

valadas commented Oct 30, 2020

Looks like MyGet is back up, firing up a new build

@valadas
Copy link
Contributor

valadas commented Oct 30, 2020

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@donker
Copy link
Contributor Author

donker commented Oct 31, 2020

I've rebased these changes on the release branch

I updated my local repo with your changes but now we have 1000+ changes ... Maybe you'll need to rebase again.

Copy link
Contributor

@valadas valadas left a 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...

@valadas
Copy link
Contributor

valadas commented Oct 31, 2020

Yeah, I am not sure how to fix that @donker @bdukes can we fix this one or we redo a new PR, I am here to help if I can do anyhting, but I am not sure how to fix this other than a cherry pick on a new PR...

@bdukes
Copy link
Contributor

bdukes commented Oct 31, 2020

I've cherry-picked from your merge commit, @donker, and then rebased again on the release branch

Copy link
Contributor

@valadas valadas left a 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.

@mitchelsellers mitchelsellers merged commit 51f5f8b into dnnsoftware:release/9.8.0 Oct 31, 2020
@donker donker deleted the fixlocalization branch November 1, 2020 09:32
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.

4 participants