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

ils: translates v0.5.0 strings #567

Merged
merged 1 commit into from
Oct 23, 2019
Merged

ils: translates v0.5.0 strings #567

merged 1 commit into from
Oct 23, 2019

Conversation

iGormilhit
Copy link

@iGormilhit iGormilhit commented Oct 17, 2019

  • NEW Translates strings for fees resources and holdings for harvested
    ebooks.
  • BETTER Harmonizes strings between api, schemas and form options.
  • FIX Fixes wrong end of line inside strings to be translated
    (javascript), in the circulation policies editor.
  • FIX Fixes typo in a string on the frontpage.

Co-Authored-by: Igor Milhit [email protected]

Why are you opening this PR?

  • Because translations weren't completely done before next release (v0.5.0).

How to test?

  • Travis is green
  • ./run-tests.sh
  • Open circulation policies editor, item_type editor and location editor and verify it's translated.

Code review check list

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

* NEW Translates strings for fees resources and holdings for harvested
  ebooks.
* BETTER Harmonizes strings between api, schemas and form options.
* FIX Fixes wrong end of line inside strings to be translated
  (javascript), in the circulation policies editor.
* FIX Fixes typo in a string on the frontpage.

Co-Authored-by: Igor Milhit <[email protected]>
@iGormilhit iGormilhit added the translations Translations label Oct 17, 2019
Copy link
Contributor

@jma jma left a comment

Choose a reason for hiding this comment

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

Do we keep the keywords NEW, BETTER, FIX in the commit message? If yes we need to update the .commitmessage.txtfile no?

@iGormilhit
Copy link
Author

@jma No, it's not anymore required. I kept the habit, though, as it's not forbidden. 🙂

@iGormilhit iGormilhit merged commit c1f5e9f into rero:dev Oct 23, 2019
@iGormilhit iGormilhit deleted the translations-v0.5.0 branch November 19, 2019 07:20
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.

2 participants