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

loans: fix automatic item assignment on pending loans #958

Closed

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Apr 27, 2020

Fixes problem when the checked-in item will be assigned to all
pending loans of its document.

This PR needs rero/invenio-circulation/baa-1410-assign-item and
rero/invenio-circulation/item_at_desk_temp

Co-Authored-by: Aly Badr aly.badr@rero.ch

Why are you opening this PR?

https://tree.taiga.io/project/rero21-reroils/task/1410?kanban-status=1224893

How to test?

It is normal that travis tests fail here, only code review is required

Code review check list

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

Fixes problem when the checked-in item will be assigned to all
pending loans of its document.

This PR needs rero/invenio-circulation/baa-1410-assign-item and
rero/invenio-circulation/item_at_desk_temp

* Fixes inveniosoftware/invenio-circulation#127.
* Removes the temporary transitions file used as a work around earlier.

Co-Authored-by: Aly Badr <aly.badr@rero.ch>
@BadrAly BadrAly self-assigned this Apr 27, 2020
@BadrAly BadrAly requested a review from blankoworld April 27, 2020 15:47
@BadrAly BadrAly closed this Apr 28, 2020
@BadrAly BadrAly deleted the baa-1410-assign-item branch November 5, 2020 13: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.

1 participant