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

translation: fix user message when an item is requested #591

Merged
merged 1 commit into from
Oct 31, 2019

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Oct 29, 2019

Co-Authored-by: Renaud Michotte renaud.michotte@gmail.com

Additional informations

  • string translations done in Transifex
  • "Github" typo fix into frontpage template ('H' should be in uppercase)

How to test?

  • Logged as an basic user
  • Ensure you choose an other language than "English"
  • Search about a document that can be requested
  • Request the item.
  • The flash message top/right of the screen should be displayed in the interface language

Code review check list

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

@zannkukai zannkukai requested review from iGormilhit, jma, rerowep and a team October 29, 2019 13:09
@zannkukai zannkukai force-pushed the ucl-#1108-translation-item-requested branch from b5d7c3d to db50493 Compare October 29, 2019 20:23
@zannkukai zannkukai self-assigned this Oct 29, 2019
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.

Commit message:

  • no uppercase in the commit message title
  • verb at the third singular person in the commit message body (fixes)
    Thanks.

* Fixes translation string "the item has been requested" with variable item_id.
* Closes rero#404.

Co-Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the ucl-#1108-translation-item-requested branch from db50493 to 50d0a19 Compare October 30, 2019 07:44
@@ -2477,98 +2477,58 @@ msgstr ""
msgid "Software under development"
msgstr ""

#: rero_ils/templates/rero_ils/frontpage.html:78
Copy link
Contributor

Choose a reason for hiding this comment

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

This does not come from your PR. It seems that translations are not up to date. Can you check that we do not loose translations from transifex. You have similar cases latter in this file.

@jma
Copy link
Contributor

jma commented Oct 31, 2019

You can the githup spelling correction in you commit message.

@zannkukai zannkukai changed the title translation: Fix user message when an item is requested translation: fix user message when an item is requested Oct 31, 2019
@jma jma merged commit ed52c18 into rero:dev Oct 31, 2019
@zannkukai zannkukai deleted the ucl-#1108-translation-item-requested branch November 6, 2019 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants