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

perf(bettermarks_proxy): drop translations subdomain #122

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

karfau
Copy link

@karfau karfau commented Sep 30, 2024

Description

Since translation files are now served from the apps domain directly.
We are still seeing some low number of requests towards the translations domain, but we already deployed the the frontend that is no longer using it a week ago.
Users affected by this will either not notice or just need to reload the page.

Links to Tickets or other pull requests

https://bettermarks.atlassian.net/browse/OT-1

Changes

Drops the translations subdomain from the list of proxied subdomains.

Datasecurity

Deployment

Bettermarks is going to shut down translations.bettermarks.com in the evening of the 30th of September 2024.

New Repos, NPM pakages or vendor scripts

None

Approval for review

  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

Since translation files are now served from the `apps` domain directly.
We are still seeing some low number of requests towards the translations domain, but we already deployed the the frontend that is no longer using it a week ago.
Users affected by this will either not notice or just need to reload the page.

https://bettermarks.atlassian.net/browse/OT-1
@karfau karfau changed the title perf: drop translations proxy perf(bettermarks_proxy): drop translations subdomain Sep 30, 2024
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.

1 participant