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

items: display items notes #278

Merged
merged 1 commit into from
Jun 20, 2020
Merged

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Jun 5, 2020

Adapts the item detail view to display notes related to an item.

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

Why are you opening this PR?

https://tree.taiga.io/project/rero21-reroils/us/1352?milestone=266303

How to test?

use rero/rero-ils#1026 (backend) bootstrap + setup

  • Log as a librarian
  • Edit an item to add a note a each type
  • Display the item detail view

image

Code review check list

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

@zannkukai zannkukai marked this pull request as ready for review June 8, 2020 06:31
@zannkukai zannkukai force-pushed the zan-#items-notes-display branch 5 times, most recently from ad26717 to 266074d Compare June 15, 2020 12:26
Adapts the item detail view to display notes related to an item.
Adds manual translations for note type.

Co-authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai merged commit ba81cb7 into rero:dev Jun 20, 2020
@zannkukai zannkukai deleted the zan-#items-notes-display branch June 20, 2020 07:37
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.

6 participants