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

removed languages not provided in dnnpages plugin #4431

Merged

Conversation

skamphuis
Copy link
Contributor

In the upgraded dnnpages-plugin for CKEditor, I left the full list of langues. Since only 3 (en,de,pl) language files are actually supported currently, the others result in an error.

This simple removal provides the same solution we had before.

I skipped creating an issue this time, since it's not in a RC yet, anyway. If needed, I'm happy to create one, of course.

@valadas
Copy link
Contributor

valadas commented Jan 24, 2021

If you can guide me to what is needed to fix languages I could fix fr or maybe some others...

@valadas
Copy link
Contributor

valadas commented Jan 24, 2021

Oh, I should have looked before asking 😄 I added french...

@donker
Copy link
Contributor

donker commented Jan 24, 2021

Can you add Dutch to this, @skamphuis ? Since we support that language quite well ...

@skamphuis
Copy link
Contributor Author

Sure, will do that tomorrow. I had that, to be honest, but found it a bit "selfish" and was thinking about a solution to make all of the editor provider localizable.

Copy link
Contributor

@donker donker 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

Copy link
Contributor

@mtrutledge mtrutledge 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

@mtrutledge mtrutledge merged commit 289e55c into dnnsoftware:develop Jan 26, 2021
@skamphuis skamphuis deleted the feature/CKEditor-dnnpages-lang branch November 23, 2022 10:39
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.

None yet

4 participants