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

request: fix pickup location update #258

Merged
merged 1 commit into from
Apr 28, 2020

Conversation

AoNoOokami
Copy link
Contributor

@AoNoOokami AoNoOokami commented Apr 28, 2020

Co-Authored-by: Alicia Zangger alicia.zangger@rero.ch

Why are you opening this PR?

To fix pickup location update on item detail view using Chrome.
To close rero/rero-ils#960.

How to test?

  1. Go to an item detail view with request.
  2. Change pickup location.

Code review check list

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

@AoNoOokami AoNoOokami self-assigned this Apr 28, 2020
@AoNoOokami AoNoOokami changed the title Zaa fix pickup location update request: fix pickup location update Apr 28, 2020
@AoNoOokami AoNoOokami force-pushed the zaa-fix-pickup-location-update branch 2 times, most recently from 115e203 to e4e95b2 Compare April 28, 2020 10:05
Copy link

@sebdeleze sebdeleze left a comment

Choose a reason for hiding this comment

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

I left some comments, do what you want with them.

Copy link
Contributor

@zannkukai zannkukai left a comment

Choose a reason for hiding this comment

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

No more remarks than Sebastien

* Fixes pickup location update on item detail view  using Chrome.
* Changes cancel button design according to UX charter.
* Adds a message on mouse hover over cancel button and no-editable pickup location.
* Closes rero/rero-ils#960.

Co-Authored-by: Alicia Zangger <alicia.zangger@rero.ch>
@AoNoOokami AoNoOokami force-pushed the zaa-fix-pickup-location-update branch from e4e95b2 to d63f0e1 Compare April 28, 2020 12:42
@AoNoOokami AoNoOokami merged commit 4688420 into rero:dev Apr 28, 2020
@AoNoOokami AoNoOokami added this to the release: v0.8.0 (rero-ils) milestone Apr 28, 2020
@AoNoOokami AoNoOokami deleted the zaa-fix-pickup-location-update branch June 2, 2020 09:22
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.

None yet

4 participants