Closed
Bug 1054902
Opened 10 years ago
Closed 10 years ago
Put Gaia Keyboard Dictionaries on fxos.cdn.mozilla.net
Categories
(Cloud Services :: Operations: Deployment Requests - DEPRECATED, task)
Cloud Services
Operations: Deployment Requests - DEPRECATED
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: timdream, Assigned: mostlygeek)
References
Details
With bug 1029951 Gaia Keyboard will come with ability to download dictionaries if they are not preloaded into the device. We should put these dictionaries on our CDN instead of pull the files from Github.
The files are *.dict files in
https://github.com/mozilla-b2g/gaia/tree/master/apps/keyboard/js/imes/latin/dictionaries
I am not sure if there is any rules on permalinks on CDN, though I think there should be a version number "1" in there so we will always be able serve files to old phones should the file format changes.
(There is a version number in the header here:
https://github.com/mozilla-b2g/gaia/blob/master/apps/keyboard/js/imes/latin/predictions.js#L173-L178 )
Reporter | ||
Comment 1•10 years ago
|
||
Benson,
I filed this bug under this component because you say so on bug 1054281. However it seems there is no default CC nor assignee on this component. Could you take this bug to the right person? Thanks!
Flags: needinfo?(bwong)
Assignee | ||
Comment 2•10 years ago
|
||
It's not a problem putting these into the CDN though I'm not sure what permanent paths you want for them.
Maybe the best thing to do is to create a new github repo for fxos.cdn.mozilla.net on github. Then when there are any changes you/whomever can make the necessary commits. I can checkout / deploy them.
What do you think?
Flags: needinfo?(bwong) → needinfo?(timdream)
Reporter | ||
Comment 3•10 years ago
|
||
Yeah that make prefect sense. I was actually a bit surprised when Kevin said we need to file bugs to make someone else change the files!
Shall we do something like https://github.com/mozilla-b2g/fxos.cdn.mozilla.net ?
Flags: needinfo?(timdream)
Assignee | ||
Comment 4•10 years ago
|
||
Looks good to me. Can you create that repo?
Reporter | ||
Comment 5•10 years ago
|
||
Repo created. Please neednifo on what you need from me when it's ready.
Assignee | ||
Comment 6•10 years ago
|
||
OK repo updated with `sync.sh` script.
Now somebody needs to update the repo with the Gaia keyboard dictionaries. If you want them to be permalinks containing the version I would suggest putting them under:
public/dictionaries/1/*
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(timdream)
Reporter | ||
Comment 7•10 years ago
|
||
Done in
https://github.com/mozilla-b2g/fxos.cdn.mozilla.net/commit/63e7244c225bebac899596ef59b614501a2cffcc
Thanks!
Flags: needinfo?(timdream)
Assignee | ||
Comment 8•10 years ago
|
||
cdn updated.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•10 years ago
|
Assignee: nobody → bwong
You need to log in
before you can comment on or make changes to this bug.
Description
•