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

circulation : toastr message when item goes in transit #339

Merged
merged 1 commit into from
Aug 25, 2020

Conversation

zannkukai
Copy link
Contributor

When an item is checked-in and goes in transit, add a toastr message to
informs user about this spec.

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

How to test?

  • Checkin an item to goes in transit. Use the SCENARIO B

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 self-assigned this Aug 17, 2020
@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch from f83f278 to b5d23c8 Compare August 17, 2020 10:17
@zannkukai zannkukai marked this pull request as ready for review August 17, 2020 11:49
@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch 2 times, most recently from 7d3eca2 to 118c208 Compare August 17, 2020 12:25
@zannkukai zannkukai changed the title circulation : toasr message when item goes in transit circulation : toastr message when item goes in transit Aug 17, 2020
Copy link
Contributor

@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: from When an item is checked-in and goes in transit, add a toastr message to informs user about this spec. to Adds a toastr message to inform the librarian that the checked-in item goes in transit. Or something else, but this spec is a bit strange, at least for me.

@iGormilhit iGormilhit added this to the v0.12.0 (rero-ils) milestone Aug 18, 2020
@iGormilhit iGormilhit added the f: circulation Concerns the circulation interface or backend label Aug 18, 2020
@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch 2 times, most recently from 4b99a16 to f0170c7 Compare August 18, 2020 13:07
Copy link
Contributor

@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 approved.

@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch 3 times, most recently from b665838 to f19d8b3 Compare August 19, 2020 12:14
@zannkukai zannkukai changed the base branch from US-fix-circulation-problems to dev August 19, 2020 12:15
@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch from f19d8b3 to cb38fcc Compare August 24, 2020 06:59
Adds a toastr message to inform the librarian that the checked-in item
goes in transit with destination informations into.

Co-Authored-by: Renaud Michtote <renaud.michotte@gmail.com>
@zannkukai zannkukai force-pushed the zan-toastr-message-for-transit branch from 5a64f15 to a653853 Compare August 24, 2020 16:05
@zannkukai zannkukai merged commit f81d36c into rero:dev Aug 25, 2020
@zannkukai zannkukai deleted the zan-toastr-message-for-transit branch September 2, 2020 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
f: circulation Concerns the circulation interface or backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants