Closed dhruvkb closed 1 year ago
Storybook and Tailwind configuration previews: Ready
Storybook: https://wordpress.github.io/openverse-frontend/_preview/2188 Tailwind: https://wordpress.github.io/openverse-frontend/_preview/2188/tailwind
Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again.
You can check the GitHub pages deployment action list to see the current status of the deployments.
Size Change: -25.6 kB (-3%)
Total Size: 882 kB
Filename | Size | Change | |
---|---|---|---|
./.nuxt/dist/client/app.js |
141 kB | -10.4 kB (-7%) | ✅ |
./.nuxt/dist/client/app.modern.js |
116 kB | -8.91 kB (-7%) | ✅ |
./.nuxt/dist/client/commons/app.js |
87.7 kB | -79 B (0%) | |
./.nuxt/dist/client/commons/app.modern.js |
78.1 kB | -77 B (0%) | |
./.nuxt/dist/client/components/v-all-results-grid.js |
7.46 kB | -86 B (-1%) | |
./.nuxt/dist/client/components/v-all-results-grid.modern.js |
4.96 kB | -90 B (-2%) | |
./.nuxt/dist/client/components/v-audio-track.js |
5.2 kB | -75 B (-1%) | |
./.nuxt/dist/client/components/v-audio-track.modern.js |
5.15 kB | -72 B (-1%) | |
./.nuxt/dist/client/components/v-external-search-form.js |
1.91 kB | -156 B (-8%) | ✅ |
./.nuxt/dist/client/components/v-external-search-form.modern.js |
1.9 kB | -147 B (-7%) | ✅ |
./.nuxt/dist/client/components/v-full-layout.js |
1.51 kB | -77 B (-5%) | ✅ |
./.nuxt/dist/client/components/v-full-layout.modern.js |
1.51 kB | -75 B (-5%) | ✅ |
./.nuxt/dist/client/components/v-home-gallery.js |
4.79 kB | -14 B (0%) | |
./.nuxt/dist/client/components/v-image-carousel.js |
4.73 kB | -34 B (-1%) | |
./.nuxt/dist/client/components/v-image-carousel.modern.js |
4.7 kB | -24 B (-1%) | |
./.nuxt/dist/client/components/v-image-grid.js |
4.85 kB | -78 B (-2%) | |
./.nuxt/dist/client/components/v-image-grid.modern.js |
2.39 kB | -77 B (-3%) | |
./.nuxt/dist/client/components/v-old-homepage-content.js |
0 B | -1.88 kB (removed) | 🏆 |
./.nuxt/dist/client/components/v-old-homepage-content.modern.js |
0 B | -1.85 kB (removed) | 🏆 |
./.nuxt/dist/client/components/v-related-images.modern.js |
2.95 kB | -79 B (-3%) | |
./.nuxt/dist/client/components/v-search-grid.js |
5.74 kB | -134 B (-2%) | |
./.nuxt/dist/client/components/v-search-grid.modern.js |
5.66 kB | -141 B (-2%) | |
./.nuxt/dist/client/components/v-skip-to-content-container.js |
911 B | +24 B (+3%) | |
./.nuxt/dist/client/components/v-skip-to-content-container.modern.js |
918 B | +22 B (+2%) | |
./.nuxt/dist/client/components/v-sources-table.js |
16.1 kB | -512 B (-3%) | |
./.nuxt/dist/client/components/v-sources-table.modern.js |
16.1 kB | -506 B (-3%) | |
./.nuxt/dist/client/pages/about.js |
1.49 kB | -31 B (-2%) | |
./.nuxt/dist/client/pages/about.modern.js |
1.49 kB | -33 B (-2%) | |
./.nuxt/dist/client/pages/audio/_id/index.js |
8.39 kB | +388 B (+5%) | 🔍 |
./.nuxt/dist/client/pages/audio/_id/index.modern.js |
5.25 kB | +381 B (+8%) | 🔍 |
./.nuxt/dist/client/pages/external-sources.js |
1.51 kB | -30 B (-2%) | |
./.nuxt/dist/client/pages/external-sources.modern.js |
1.51 kB | -27 B (-2%) | |
./.nuxt/dist/client/pages/feedback.js |
1.29 kB | -29 B (-2%) | |
./.nuxt/dist/client/pages/feedback.modern.js |
1.29 kB | -29 B (-2%) | |
./.nuxt/dist/client/pages/image/_id/index.js |
9.7 kB | +382 B (+4%) | |
./.nuxt/dist/client/pages/image/_id/index.modern.js |
7.7 kB | +255 B (+3%) | |
./.nuxt/dist/client/pages/image/_id/report.js |
4.22 kB | +629 B (+18%) | ⚠️ |
./.nuxt/dist/client/pages/image/_id/report.modern.js |
4.67 kB | +425 B (+10%) | ⚠️ |
./.nuxt/dist/client/pages/index.js |
6.85 kB | -1.8 kB (-21%) | 🎉 |
./.nuxt/dist/client/pages/index.modern.js |
6.77 kB | -1.75 kB (-20%) | 🎉 |
./.nuxt/dist/client/pages/privacy.js |
963 B | -32 B (-3%) | |
./.nuxt/dist/client/pages/privacy.modern.js |
966 B | -29 B (-3%) | |
./.nuxt/dist/client/pages/search-help.js |
1.56 kB | -62 B (-4%) | |
./.nuxt/dist/client/pages/search-help.modern.js |
1.53 kB | -89 B (-6%) | ✅ |
./.nuxt/dist/client/pages/search.js |
5.05 kB | -126 B (-2%) | |
./.nuxt/dist/client/pages/search.modern.js |
2.55 kB | -123 B (-5%) | ✅ |
./.nuxt/dist/client/pages/search/audio.js |
6.07 kB | -56 B (-1%) | |
./.nuxt/dist/client/pages/search/audio.modern.js |
3.58 kB | -42 B (-1%) | |
./.nuxt/dist/client/pages/search/image.js |
563 B | -33 B (-6%) | ✅ |
./.nuxt/dist/client/pages/search/image.modern.js |
2.63 kB | -94 B (-3%) | |
./.nuxt/dist/client/pages/search/index.js |
508 B | +28 B (+6%) | 🔍 |
./.nuxt/dist/client/pages/search/index.modern.js |
515 B | +30 B (+6%) | 🔍 |
./.nuxt/dist/client/pages/sources.js |
1.52 kB | -34 B (-2%) | |
./.nuxt/dist/client/pages/sources.modern.js |
1.52 kB | -39 B (-2%) | |
./.nuxt/dist/client/vendors/app.js |
63.8 kB | +47 B (0%) | |
./.nuxt/dist/client/vendors/app.modern.js |
63 kB | -106 B (0%) | |
./.nuxt/dist/client/components/v-modal.js |
991 B | +991 B (new file) | 🆕 |
./.nuxt/dist/client/components/v-modal.modern.js |
981 B | +981 B (new file) | 🆕 |
Success update:
With the right secrets in the actions, all 200 translations are being downloaded...
...and incorporated into the Docker image.
@zackkrida I'm very bad with JS Promises, even getting this PR to this usable state was very difficult for me. If you can push those changes for improved logging or fallback, that'll be very helpful.
I'm considering refactoring these scripts (and maybe use TypeScript) when I have some free time because it's very difficult to work with Promise chains, at least for me if not anyone else.
@dhruvkb sure, I can make some suggestions later today.
@zackkrida my grasp on Promises was holding me back, I replaced all of that with async
/await
in #2192. Will merge that into this PR if you agree with those changes.
Fixes
Fixes #2184 by @dhruvkb
Description
This PR adds bulk downloading for translations by logging into GlotPress, downloading the ZIP and extracting the JSON content. It reuses the existing framework for massaging them from
jed1x
to Vue i18n's format and then writing them to the destination dir.If GlotPress credentials are not present in the environment, the old throttled approach will be used as the fallback, a Make WordPress membership is not needed for development.
Other improvements
la.json
) too. Unfortuanately we can't use it because it's not in the valid locales list.locales-list.json
that was not being used anywhere and had no references pointing to it.Testing Instructions
You will see the parallelised download that might throttle.
You will see the bulk download that does not throttle.
Checklist
Update index.md
).main
) or a parent feature branch.[best_practices]: https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project#_commit_guidelines
Developer Certificate of Origin
Developer Certificate of Origin
``` Developer Certificate of Origin Version 1.1 Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129 Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed. Developer's Certificate of Origin 1.1 By making a contribution to this project, I certify that: (a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or (b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or (c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it. (d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved. ```