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

Backport from edge: added support for language locales in apps #3303

Merged
merged 1 commit into from
Oct 7, 2022

Conversation

glpatcern
Copy link
Member

@glpatcern glpatcern commented Oct 3, 2022

The goal here is to backport the minimal logic from edge to support different locales in WOPI apps, and drive this by configuration rather than by query parameters.

In addition, the DC_LLCC parameter should be used for Office 365 to compensate issues with incorrect dates computations depending on the locale:

https://learn.microsoft.com/en-us/microsoft-365/cloud-storage-partner-program/online/discovery#dc_llcc

@glpatcern glpatcern self-assigned this Oct 3, 2022
@update-docs
Copy link

update-docs bot commented Oct 3, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@glpatcern glpatcern requested a review from a team as a code owner October 3, 2022 12:31
@glpatcern glpatcern marked this pull request as draft October 4, 2022 06:39
@glpatcern glpatcern force-pushed the lang-supp-in-app branch 3 times, most recently from 02bd43c to f979ea4 Compare October 4, 2022 10:38
@glpatcern glpatcern marked this pull request as ready for review October 4, 2022 12:22
@glpatcern glpatcern marked this pull request as draft October 5, 2022 16:52
@glpatcern glpatcern force-pushed the lang-supp-in-app branch 2 times, most recently from 211f85a to e155d9d Compare October 5, 2022 20:25
@glpatcern glpatcern marked this pull request as ready for review October 5, 2022 20:26
@glpatcern
Copy link
Member Author

@micbar as you authored #3291 you may want to see a different flavor for it:

  • here we have based this on server-side configuration rather than client-side (opaque) parameters: we plan to force a locale in our deployment, otherwise the default behavior is autodetection by the application engine.
  • for Office365, the parameters are ui=<UI_LLCC> and rs=<DC_LLCC>. <UI_LLCC> is a "placeholder", the query parameter names are to be taken from the /hosting/discovery XML endpoint.

@glpatcern glpatcern merged commit 1e4948b into cs3org:master Oct 7, 2022
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 18, 2022
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 18, 2022
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 20, 2022
vascoguita pushed a commit to vascoguita/reva that referenced this pull request Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants