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: adapt reroils #955

Merged

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Apr 26, 2020

Adapts reroils after the upgrade to invenio-circulation v1.0.0a21.

The two transitions ItemOnLoanToItemReturned and ItemOnLoanToItemReturned are
temporarily updated and inserted into reroils until a fix for the issue
inveniosoftware/invenio-circulation#127 is available.

This PR expects the fix for the ITEM_AT_DESK problem in the current version
of invenio-circulation.

Some timezone units testing are disabled until a fix is given in a later PR.

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

Dependencies

This PR requires the rero/invenio-circulation.item_at_desk branch

Why are you opening this PR?

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

How to test?

bootstrap + run_tests.sh

Code review check list

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

@BadrAly BadrAly force-pushed the baa-1431-adapt-reroils branch from 4fb8568 to 86d6bac Compare April 26, 2020 07:20
@BadrAly BadrAly added the f: circulation Concerns the circulation interface or backend label Apr 26, 2020
@BadrAly BadrAly self-assigned this Apr 26, 2020
@BadrAly BadrAly force-pushed the baa-1431-adapt-reroils branch 2 times, most recently from 722005d to 5dc5b74 Compare April 26, 2020 14:26
@BadrAly BadrAly marked this pull request as ready for review April 27, 2020 06:13
rero_ils/config.py Outdated Show resolved Hide resolved
rero_ils/modules/items/api.py Outdated Show resolved Hide resolved
rero_ils/modules/items/api.py Outdated Show resolved Hide resolved
rero_ils/modules/items/api.py Outdated Show resolved Hide resolved
rero_ils/modules/items/api.py Outdated Show resolved Hide resolved
rero_ils/modules/items/api_views.py Outdated Show resolved Hide resolved
rero_ils/modules/loans/transitions.py Outdated Show resolved Hide resolved
rero_ils/modules/loans/transitions.py Outdated Show resolved Hide resolved
rero_ils/modules/loans/transitions.py Outdated Show resolved Hide resolved
rero_ils/modules/loans/utils.py Outdated Show resolved Hide resolved
rero_ils/modules/patrons/api.py Outdated Show resolved Hide resolved
@BadrAly BadrAly force-pushed the baa-1431-adapt-reroils branch from 5dc5b74 to b0bfecf Compare April 27, 2020 09:17
@BadrAly BadrAly requested a review from zannkukai April 27, 2020 09:22
rero_ils/modules/loans/listener.py Outdated Show resolved Hide resolved
Adapts reroils after the upgrade to invenio-circulation v1.0.0a21.

The two transitions ItemOnLoanToItemReturned and ItemOnLoanToItemReturned are
temporarily updated and inserted into reroils until a fix for the issue
inveniosoftware/invenio-circulation#127 is available.

This PR expects the fix for the ITEM_AT_DESK problem in the current version
of invenio-circulation.

Some timezone units testing are disabled until a fix is given in a later PR.

Co-Authored-by: Aly Badr <aly.badr@rero.ch>
@BadrAly BadrAly force-pushed the baa-1431-adapt-reroils branch from b0bfecf to 4c4c1c9 Compare April 27, 2020 09:51
@BadrAly BadrAly merged commit e6ab9c3 into rero:US1394-invenio-circulation Apr 27, 2020
@BadrAly BadrAly deleted the baa-1431-adapt-reroils 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
f: circulation Concerns the circulation interface or backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants