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: update request permissions #343

Merged

Conversation

AoNoOokami
Copy link
Contributor

@AoNoOokami AoNoOokami commented Aug 21, 2020

  • Updates permissions to allow to cancel a request and to update request pickup location when an item is in transit for pickup.

Co-Authored-by: Alicia Zangger alicia.zangger@rero.ch

Why are you opening this PR?

Task 1685 of US 1665

Dependencies

My PR depends on rero-ils#<xx>'s PR(s):

How to test?

Try to change the request pickup location of an item that is in transit for pickup.

Code review check list

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

@AoNoOokami AoNoOokami self-assigned this Aug 21, 2020
@AoNoOokami AoNoOokami added the f: circulation Concerns the circulation interface or backend label Aug 21, 2020
@AoNoOokami AoNoOokami force-pushed the zaa-#1685-permissions-cancel-update-request branch from 021870d to b212aa6 Compare August 21, 2020 08:45
@AoNoOokami AoNoOokami added this to the v0.12.0 (rero-ils) milestone Aug 21, 2020
@AoNoOokami AoNoOokami marked this pull request as ready for review August 21, 2020 13:15
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.

Format the commit message body with a hard wrap at column 72. Thanks.

* Updates permissions to allow to cancel a request and to update request
pickup location when an item is in transit for pickup.

Co-Authored-by: Alicia Zangger <alicia.zangger@rero.ch>
@AoNoOokami AoNoOokami force-pushed the zaa-#1685-permissions-cancel-update-request branch from b212aa6 to adbf48e Compare August 24, 2020 08:39
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.

@AoNoOokami AoNoOokami merged commit 588821e into rero:dev Aug 24, 2020
@AoNoOokami AoNoOokami deleted the zaa-#1685-permissions-cancel-update-request branch September 15, 2020 11:44
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