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

items: fix online locations status #580

Merged
merged 1 commit into from
Oct 28, 2019

Conversation

zannkukai
Copy link
Contributor

@zannkukai zannkukai commented Oct 24, 2019

Co-Authored-by: Renaud Michotte [email protected]

How to test?

  • run scripts/setup
  • run curl -XPOST -H 'Content-Type: application/json' "http://localhost:9200/locations/_search?pretty" -d '{"query": {"term": { "is_online": true } } } }' to get online location pids. Result must be [12,18,4]
  • run curl -XPOST -H 'Content-Type: application/json' "http://localhost:9200/items/_search?pretty" -d '{"query": {"terms": { "location.pid": [12, 18, 4] } } }' to get number if items in these locations. The number should be "0"

Code review check list

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

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.

For the commit message:

  • title: explain what this commit does, not what was wrong
  • message: use sentence, with verbs, and there you can explain why this is necessary
  • do not forget Closes rero/rero-ils#<issue-pid>

Thanks !

@zannkukai zannkukai force-pushed the ucl-#1096-online-item-type branch from 83ed238 to 263309f Compare October 24, 2019 10:00
@zannkukai zannkukai changed the title items: physical items cannot be linked with Online location items: fix online locations status Oct 24, 2019
@zannkukai zannkukai requested a review from iGormilhit October 24, 2019 10:02
@zannkukai zannkukai changed the base branch from master to dev October 24, 2019 10:34
@zannkukai zannkukai closed this Oct 24, 2019
@zannkukai zannkukai reopened this Oct 24, 2019
@zannkukai zannkukai force-pushed the ucl-#1096-online-item-type branch 3 times, most recently from 58fd5f4 to 9bd9e04 Compare October 24, 2019 19:43
Copy link
Contributor

@AoNoOokami AoNoOokami left a comment

Choose a reason for hiding this comment

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

Curl requests didn't work on localhost:9200, so I tested with Kibana on localhost:5601. The results are the expected ones.

* Fix `data/location.json` file : Removing not online location name `Online`
* Closes rero#573

Co-Authored-by: Renaud Michotte <[email protected]>
@zannkukai zannkukai force-pushed the ucl-#1096-online-item-type branch from 9bd9e04 to e7efa6c Compare October 28, 2019 09:51
@zannkukai zannkukai merged commit 05f20d9 into rero:dev Oct 28, 2019
@zannkukai zannkukai deleted the ucl-#1096-online-item-type branch November 6, 2019 07: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.

3 participants