Booking: port Capture Resources screen to Angular

Bug #1851306 reported by Jane Sandberg
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Wishlist
Unassigned

Bug Description

All of the booking module is now in Angular, except for the dojo Capture Resources screen. This screen was not part of bug 1816475, since it needed to be able to print a hold slip for the captured resource.

Now that we have server-managed print templates in Angular (bug 1825851), we can port the Capture Resources screen too. "Capture Resources" might also make a good action for some grids in other booking screens, like Manage Reservations or Pick Up Reservations.

I will tentatively plan to do this for 3.5, but if somebody else wants to take this on, please feel free to un-assign me.

Revision history for this message
Jane Sandberg (sandbergja) wrote :

Here's a branch that ports the capture resources screen to Angular: user/sandbergja/lp1851306_port_booking_capture_to_angular

Here's a link: https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/sandbergja/lp1851306_port_booking_capture_to_angular

Let me know what you think!

tags: added: pullrequest
Changed in evergreen:
assignee: Jane Sandberg (sandbej) → nobody
Revision history for this message
Bill Erickson (berick) wrote :

Hi Jane, took a very quick look at this today. 2 things caught my eye, a few untranslatable toast strings and a JS error that is consistently logged on page load:

CaptureComponent.html:10 ERROR TypeError: this.reloadGrid is not a function
    at ReservationsGridComponent.ngOnChanges (reservations-grid.component.ts:299)

Revision history for this message
Jane Sandberg (sandbergja) wrote :

Thanks for catching those, Bill. I force-pushed fixes to those issues to the same branch.

Revision history for this message
Terran McCanna (tmccanna) wrote :
tags: added: signedoff
Changed in evergreen:
milestone: 3.5-beta → 3.5.0
Galen Charlton (gmc)
Changed in evergreen:
assignee: nobody → Galen Charlton (gmc)
milestone: 3.5.0 → 3.6-beta
Revision history for this message
Galen Charlton (gmc) wrote :

Pushed to master for inclusion in 3.6 along with a couple follow-ups. Thanks, Jane and Terran!

Changed in evergreen:
status: New → Fix Committed
assignee: Galen Charlton (gmc) → nobody
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.