Closed Bug 1376454 Opened 8 years ago Closed 6 years ago

Remove git submodule to pontoon-intro from the pontoon repository

Categories

(Webtools Graveyard :: Pontoon, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: adrian, Assigned: mathjazz)

Details

Attachments

(1 file)

Currently pontoon uses a git submodule to load the code for the intro page into the repository. Submodules are generally hard to use and a bad practice, so we should replace with something else. Plan is as follows: 1. move pontoon-intro code and history into the pontoon repo, _except_ for the content of the ``static/locales``; 2. move the content of ``static/locales`` to the root of the pontoon-intro repo, and remove everything else.
Priority: -- → P3
How about we instead move the content and history of ``static/locales`` into a new repository? That way we have an easier fallback (the current pontoon-intro repo), and we can run some tests.
(In reply to Adrian Gaudebert [:adrian] from comment #1) > How about we instead move the content and history of ``static/locales`` into > a new repository? That way we have an easier fallback (the current > pontoon-intro repo), and we can run some tests. SGTM.
Here's step one: I have moved the entire ``static/locales`` folder and its history into a new repo, https://github.com/adngdb/pontoon-intro-locales The next step would be to take the code, without the locales, of pontoon-intro and move it into the pontoon repo. We don't care about the git history there. Now the question is: how do we import the locales into the pontoon website? We don't want to replace the submodule with another one, so I think we'll want to add a step to our build process to pull in the new pontoon-intro-locales content. Matjaz, what do you think about that? How easy would that be with the current system?
Flags: needinfo?(m)
(In reply to Adrian Gaudebert [:adrian] from comment #3) > Here's step one: I have moved the entire ``static/locales`` folder and its > history into a new repo, https://github.com/adngdb/pontoon-intro-locales LGTM. You should move it under the mozilla org. > The next step would be to take the code, without the locales, of > pontoon-intro and move it into the pontoon repo. We don't care about the git > history there. Now the question is: how do we import the locales into the > pontoon website? We don't want to replace the submodule with another one, so > I think we'll want to add a step to our build process to pull in the new > pontoon-intro-locales content. > > Matjaz, what do you think about that? How easy would that be with the > current system? We don't need to import locales into the website. We only need the website to appear on https://pontoon.mozilla.org/intro/ (in English). It's not really localized, it only serves as the source website Pontoon homepage uses for in-context l10n demo. We should however add a data migration that will point the pontoon-intro project to the new repository by updating the url, permalink_prefix and website fields.
Flags: needinfo?(m)
(In reply to Matjaz Horvat [:mathjazz] from comment #4) > We should however add a data migration that will point the pontoon-intro > project to the new repository by updating the url, permalink_prefix and > website fields. And now that projects can use DB as the string source, we can also stop using the string repository.
Assignee: adrian → m
Status: NEW → ASSIGNED
Commit pushed to master at https://github.com/mozilla/pontoon https://github.com/mozilla/pontoon/commit/6a1ec623d1518867e6b9b1d2059da00716a03ac0 Fix bug 1376454: Un-submodule pontoon.intro (#1064) And also: * Update Pontoon Intro copy to become in-context demo page. * Rename `intro` app to `in-context` and move URL config into it. * Remove submodule instructions from developer & deployment docs, and travis config.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: