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)
Webtools Graveyard
Pontoon
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.
Assignee | ||
Updated•8 years ago
|
Priority: -- → P3
Reporter | ||
Comment 1•8 years ago
|
||
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.
Assignee | ||
Comment 2•7 years ago
|
||
(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.
Reporter | ||
Comment 3•7 years ago
|
||
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)
Assignee | ||
Comment 4•7 years ago
|
||
(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)
Assignee | ||
Comment 5•7 years ago
|
||
(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 | ||
Updated•6 years ago
|
Assignee: adrian → m
Status: NEW → ASSIGNED
Comment 6•6 years ago
|
||
Comment 7•6 years ago
|
||
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.
Updated•6 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Updated•3 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•