X Tutup
Skip to content

Override browser locales with OC locale in ajax requests#38073

Merged
phil-davis merged 1 commit intomasterfrom
bugfix/e4159
Nov 4, 2020
Merged

Override browser locales with OC locale in ajax requests#38073
phil-davis merged 1 commit intomasterfrom
bugfix/e4159

Conversation

@VicDeo
Copy link
Member

@VicDeo VicDeo commented Nov 3, 2020

Description

Send Accept-Language header based on the current OC user locale

Related Issue

Motivation and Context

owncloud API uses Accept-Language header to set the response locale. This PR overrides a default Accept-Language sent by browser with the current user locale

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@DeepDiver1975 DeepDiver1975 self-requested a review November 3, 2020 13:48
@VicDeo VicDeo force-pushed the bugfix/e4159 branch 2 times, most recently from f344e10 to 343b406 Compare November 3, 2020 18:05
@owncloud owncloud deleted a comment from update-docs bot Nov 4, 2020
@phil-davis
Copy link
Contributor

There were silly drone/docker errors. I rebased to get fresh CI.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 4, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@phil-davis phil-davis merged commit 51282c0 into master Nov 4, 2020
@delete-merged-branch delete-merged-branch bot deleted the bugfix/e4159 branch November 4, 2020 04:33
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.

3 participants

X Tutup