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

Remove bilingual files removed from repository #2057

Closed
bugzilla-to-github opened this issue Mar 7, 2017 · 4 comments · Fixed by #3312
Closed

Remove bilingual files removed from repository #2057

bugzilla-to-github opened this issue Mar 7, 2017 · 4 comments · Fixed by #3312
Labels
bug P2 We want to ship it soon, possibly in the current quarter sync

Comments

@bugzilla-to-github
Copy link
Collaborator

This issue was created automatically by a script.

Bug 1345431

Bug Reporter: @mathjazz
CC: @TheoChevalier

Projects using bilingual file formats (xliff, po, lang...) can have different files exposed for each locale. If the file is added to the locale folder in repository, we detect it and enable it Pontoon. If it's removed, we also detect it, but don't remove it from Pontoon.

This doesn't happen often, but it still does:
mozilla-l10n/engagement-l10n@7bac95f

Implementations notes:
Files are represented with an instance of the TranslatedResource model. If an instance exist for the given locale, we expose the file in Pontoon. When a file is removed from the repository, we need to remove the TranslatedResource instance and aggregate denormalized stats for the corresponding Project, Locale and ProjectLocale instance.

Unlike changed_source_files(), changed_locales_files() ignores removed_files:
https://github.com/mozilla/pontoon/blob/master/pontoon/sync/vcs/models.py#L129

@bugzilla-to-github
Copy link
Collaborator Author

Comment Author: @mathjazz

*** Bug #1427767 has been marked as a duplicate of this bug. ***

@mathjazz mathjazz changed the title [sync] Remove bilingual files removed from repository Remove bilingual files removed from repository Nov 10, 2021
@mathjazz mathjazz added P1 Must be fixed immediately, drop any other work and removed P3 Default, possibly shipping in the following two quarters labels Sep 9, 2022
@mathjazz mathjazz self-assigned this Sep 9, 2022
@mathjazz
Copy link
Collaborator

mathjazz commented Sep 9, 2022

Bumping priority, because the bug might hit us again soon after we stop using project configuration for "Foundation website - content".

@flodolo
Copy link
Collaborator

flodolo commented Mar 6, 2024

@mathjazz
Is it expected to have a P1 issue open for so long?

@mathjazz
Copy link
Collaborator

mathjazz commented Mar 6, 2024

No. :)

This should be taken care of as part of the "Improve Pontoon sync reliability" OKR.

Also, P1 was probably to high priority in first place.

@mathjazz mathjazz added P2 We want to ship it soon, possibly in the current quarter and removed P1 Must be fixed immediately, drop any other work labels Mar 6, 2024
@mathjazz mathjazz moved this from Done to Moving and removing files in Pontoon Sync reliability Apr 4, 2024
@eemeli eemeli mentioned this issue Nov 27, 2024
@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in Pontoon Roadmap Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug P2 We want to ship it soon, possibly in the current quarter sync
Projects
Archived in project
Status: Moving and removing files
Development

Successfully merging a pull request may close this issue.

3 participants