Skip to content
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

translations: add translations API #997

Merged
merged 1 commit into from
May 25, 2020
Merged

translations: add translations API #997

merged 1 commit into from
May 25, 2020

Conversation

jma
Copy link
Contributor

@jma jma commented May 19, 2020

  • Adds a new HTTP endpoint to expose the python po based translations
    in JSON in order to use theses translations in single app application
    instead of duplicates the translations in several projects.
  • Fixes spelling in comment.
  • Exposes invenio-i18n to the uwsgi API.
  • Fixes tests with the invenio-i18n enabled for the API.
  • Removes useless template filters and their tests.

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch

Why are you opening this PR?

Dependencies

My PR depends on the following rero-ils-ui's PR(s):

How to test?

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@jma jma force-pushed the maj-ui-translations branch 5 times, most recently from 90f11d1 to 1480cbc Compare May 19, 2020 11:43
@jma jma requested review from rerowep and sebdeleze May 19, 2020 11:43
@jma jma marked this pull request as ready for review May 19, 2020 11:45
@iGormilhit iGormilhit added the translations Translations label May 20, 2020
@iGormilhit iGormilhit added this to the v0.9.0 milestone May 20, 2020
@iGormilhit iGormilhit self-requested a review May 20, 2020 04:33
Copy link

@iGormilhit iGormilhit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, explain why this commit it useful, not only what it does. Thanks.

@@ -82,7 +82,8 @@ def run(self):
'rero-ils = rero_ils.modules.ext:REROILSAPP'
],
'invenio_base.api_apps': [
'rero-ils = rero_ils.modules.ext:REROILSAPP'
'rero-ils = rero_ils.modules.ext:REROILSAPP',
'invenio_i18n = invenio_i18n:InvenioI18N'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You didn't listen to me ;-)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No sorry.

* Adds a new HTTP endpoint to expose the python po based translations
in JSON in order to use theses translations in single app application
instead of duplicating the translations in several projects.
* Fixes spelling in comment.
* Exposes `invenio-i18n` to the uwsgi backend application.
* Fixes tests with the `invenio-i18n` enabled for the backend application.
* Removes useless template filters and their tests.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma force-pushed the maj-ui-translations branch from e485c5b to 4cbdff8 Compare May 25, 2020 10:08
@jma jma merged commit b9863e3 into rero:dev May 25, 2020
@jma jma deleted the maj-ui-translations branch January 13, 2022 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translations Translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants