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: checkin operations when item/patron barcode are identical #776

Merged
merged 1 commit into from
Dec 23, 2021

Conversation

Garfield-fr
Copy link
Contributor

@Garfield-fr Garfield-fr commented Dec 22, 2021

A dialog appears when there is an item and a patron with the same barcode.
The library chooses the action to execute.

Co-Authored-by: Bertrand Zuchuat bertrand.zuchuat@rero.ch

Code review check list

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

@Garfield-fr Garfield-fr marked this pull request as draft December 22, 2021 10:56
@github-actions github-actions bot added f: circulation Concerns the circulation interface or backend f: professional ui Professional interface labels Dec 22, 2021
@Garfield-fr Garfield-fr force-pushed the zub-circulation-feature branch 8 times, most recently from 107bbac to 3a1daf0 Compare December 22, 2021 13:47
A dialog appears when there is an item and a patron with the same barcode.
The library chooses the action to execute.

* Closes rero/rero-ils#2597.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
@Garfield-fr Garfield-fr marked this pull request as ready for review December 22, 2021 14:06
@jma jma requested review from jma and removed request for zannkukai December 23, 2021 09:20
You should have received a copy of the GNU Affero General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>.
-->
<div class="modal show d-block" role="dialog" (window:keypress)="checkinItemOnEnterKey($event)">
Copy link
Contributor

Choose a reason for hiding this comment

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

d-block seem not required here.

</div>
<div class="modal-body">
<div class="font-weight-bold" translate>
One item and one patron were found.
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you check this message with the PO? Can be more explicit.

})
export class CheckinActionComponent {

/** Checkin action */
Copy link
Contributor

Choose a reason for hiding this comment

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

In case of patron the action is not a checkin, you can change your comment.

@jma jma merged commit 51bfea0 into rero:staging Dec 23, 2021
@Garfield-fr Garfield-fr deleted the zub-circulation-feature branch March 14, 2022 12:14
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 f: professional ui Professional interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

circulation: problem to manage checkin operations when item/patron barcode are identical
3 participants