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: display library name while transit #178

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

blankoworld
Copy link
Contributor

Co-Authored-by: Olivier DOSSMANN git@dossmann.net

Why are you opening this PR?

Because of rero/rero-ils#769

How to test?

Cf. issue 769 to check

Code review check list

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

@@ -41,7 +41,7 @@ $ git clone https://github.com/rero/rero-ils-ui.git/
$ cd rero-ils-ui
$ nvm use lts/dubnium
# Install angular-cli
$ npm i -g angular-cli
$ npm i -g @angular/cli@8.3.25
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it necessary ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In documentation, I think that yes.
I follow it and fetch angular-cli 1.0.0beta32 (or similar) which was a problem to have a development environment.

@blankoworld blankoworld force-pushed the doo-769-fix-library-name branch 2 times, most recently from 58b789c to 5a539fe Compare February 20, 2020 15:26
@Garfield-fr Garfield-fr self-requested a review February 21, 2020 05:39
* Displays library name (and location name) while loan is in transit status
* Fixes rero/rero-ils#769
* Completes documentation about angular-cli installation

Co-Authored-by: Olivier DOSSMANN <git@dossmann.net>
Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
@blankoworld blankoworld merged commit caa7465 into rero:dev Feb 21, 2020
@blankoworld blankoworld deleted the doo-769-fix-library-name branch July 17, 2020 10:19
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