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: integration of invenio-circulation APIs #136

Merged
merged 1 commit into from
Nov 7, 2018

Conversation

BadrAly
Copy link

@BadrAly BadrAly commented Nov 5, 2018

  • NEW makes invenio circ APIs available

Signed-off-by: Aly Badr aly.badr@rero.ch

To test:

  • ./scripts/bootstrap
  • ./scripts/setup
  • ./run_tests.sh

To test some circulation transactions, check examples in Taiga:
https://tree.taiga.io/project/rero21-reroils/task/708?kanban-status=1224894

@BadrAly BadrAly force-pushed the baa-#708-invenio-circ-api branch 2 times, most recently from 70f4122 to 3e4c15f Compare November 6, 2018 14:28
Copy link

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

As long as this PR will be rebased against the master branch after the integration of #135, it's ok for me. Thanks.

@reropag reropag self-requested a review November 7, 2018 10:25
@BadrAly BadrAly force-pushed the baa-#708-invenio-circ-api branch 4 times, most recently from 44649b7 to caca1e8 Compare November 7, 2018 13:30
 * NEW makes invenio circ APIs available

Signed-off-by: Aly Badr <aly.badr@rero.ch>
@BadrAly BadrAly force-pushed the baa-#708-invenio-circ-api branch from caca1e8 to 3d7a3b3 Compare November 7, 2018 13:31
@rerowep rerowep merged commit 6c5a6f1 into rero:master Nov 7, 2018
@BadrAly BadrAly deleted the baa-#708-invenio-circ-api branch January 10, 2019 09:45
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.

5 participants