-
Notifications
You must be signed in to change notification settings - Fork 46
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
editoast, front: host fonts for the map in editoast #10788
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #10788 +/- ##
=======================================
Coverage 81.93% 81.93%
=======================================
Files 1084 1085 +1
Lines 107199 107232 +33
Branches 737 737
=======================================
+ Hits 87832 87863 +31
- Misses 19326 19328 +2
Partials 41 41
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
5514494
to
2bd29b5
Compare
c5a9319
to
35865d0
Compare
fcb377b
to
7283410
Compare
0af99d4
to
ab89c1a
Compare
c0f4b22
to
8caf39b
Compare
ab89c1a
to
674f86c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice follow up of the sprite PR. Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
58d8372
to
8176887
Compare
03e78cd
to
f9f8bdd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for the frontend
(I'm not sure the per-commit split makes sense: the script and translation file changes belong to the first commit IMHO.)
The glyphs are generated during the build step Closes #10627 Signed-off-by: Tristram Gräbener <[email protected]>
Signed-off-by: Tristram Gräbener <[email protected]>
Signed-off-by: Tristram Gräbener <[email protected]>
f9f8bdd
to
7cadb58
Compare
I re-arranged the commits |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a small typo, otherwise lgtm
@@ -61,6 +61,9 @@ | |||
"electrical_profiles": { | |||
"NotFound": "Profil électrique '{{electrical_profile_set_id}}' non trouvé" | |||
}, | |||
"fonts": { | |||
"FileNotFound": "Fichier pour la police de charactère demandée non trouvé" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is a typo : caractère instead of charactère
The glyphs are generated during the build step
Closes #10627
How to test: open the map, filter network requests by
roboto
and make sure that everything is 200 and is now on localhost.(Should probably be merged after #10680 and #10628)