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

Fetch proper translations #14801

Merged
merged 1 commit into from
Mar 25, 2019
Merged

Conversation

rullzer
Copy link
Member

@rullzer rullzer commented Mar 22, 2019

Fixes #14793

This is caused by the mess we have with OC\Settings mapping to settings
and lib/private/Settings.

Anyway this is the quick fix. Moving stuff around for 17 seems better.

Signed-off-by: Roeland Jago Douma roeland@famdouma.nl

@rullzer rullzer added the 3. to review Waiting for reviews label Mar 22, 2019
@rullzer rullzer added this to the Nextcloud 16 milestone Mar 22, 2019
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Makes sense

Test might break, but @MorrisJobke's bot will tell us :)

Fixes #14793

This is caused by the mess we have with OC\Settings mapping to settings
and lib/private/Settings.

Anyway this is the quick fix. Moving stuff around for 17 seems better.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
@rullzer rullzer force-pushed the fix/14793/fetch_right_translations branch from 7b99c9d to a7fb71f Compare March 25, 2019 08:11
@nextcloud nextcloud deleted a comment from faily-bot bot Mar 25, 2019
@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 25, 2019
@rullzer rullzer merged commit 8b7fff7 into master Mar 25, 2019
@rullzer rullzer deleted the fix/14793/fetch_right_translations branch March 25, 2019 09:15
@rullzer rullzer mentioned this pull request Mar 26, 2019
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Translation in the "Sharing" administrator settings
4 participants