Bug 38010 - Migrate vendors to Vue
Summary: Migrate vendors to Vue
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: Main
Hardware: All All
: P5 - low new feature
Assignee: Matt Blenkinsop
QA Contact: Jonathan Druart
URL: https://github.com/openfifth/koha/com...
Keywords: release-notes-needed, roadmap_25_05
Depends on: 38011 38390 38941 38204
Blocks: 38207 38208 38256 38262 38899 39709 39711 39776 39785 39794 39795 39796 39797 39836 39837 39838 39858
  Show dependency treegraph
 
Reported: 2024-09-26 13:15 UTC by Matt Blenkinsop
Modified: 2025-05-16 15:20 UTC (History)
11 users (show)

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
25.05.00
Circulation function:


Attachments
Bug 38010: Add vue app and list component (31.74 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.57 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (11.38 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.28 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.83 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (46.41 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (3.99 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a nav link button component (20.98 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.75 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.54 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.80 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.04 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.48 KB, patch)
2024-11-07 11:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (3.02 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.14 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.39 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (6.25 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.02 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.32 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.53 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.17 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.18 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.32 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (16.29 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (937 bytes, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.91 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.77 KB, patch)
2024-11-07 11:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (31.78 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.57 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (11.38 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.28 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.83 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (46.41 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (3.99 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a nav link button component (21.08 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.79 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.54 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.80 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.04 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.48 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (3.02 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.14 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.40 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (6.25 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.02 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.32 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.53 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.17 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.18 KB, patch)
2024-11-07 11:56 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.32 KB, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (16.28 KB, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (937 bytes, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.91 KB, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.77 KB, patch)
2024-11-07 11:57 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (4.82 KB, patch)
2024-11-07 13:26 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (31.76 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.57 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (11.38 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.28 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.83 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (46.41 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (3.99 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a nav link button component (21.08 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.79 KB, patch)
2024-11-07 13:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.54 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.80 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.04 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.48 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (3.02 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.14 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.40 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (6.25 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.02 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.32 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.53 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.17 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.18 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.32 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (16.28 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (937 bytes, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.91 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.77 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (4.82 KB, patch)
2024-11-07 13:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add recevie shipments action to table (2.50 KB, patch)
2024-11-07 15:24 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add receive shipments action to table (2.50 KB, patch)
2024-11-08 09:58 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Restore new basket/contract buttons (1.56 KB, patch)
2024-11-08 09:58 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add dropdown button component (4.48 KB, patch)
2024-11-08 13:09 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Reset basket view (18.18 KB, patch)
2024-11-08 13:09 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Reset basket view (21.16 KB, patch)
2024-11-08 14:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (31.76 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.57 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (11.38 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.28 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.83 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (46.41 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (3.99 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a nav link button component (21.27 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.79 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.54 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.80 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.04 KB, patch)
2024-11-11 14:59 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.48 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (3.02 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.14 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.03 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (6.25 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.02 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.32 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.53 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.17 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.18 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.32 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (16.27 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (937 bytes, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.91 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.77 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (5.50 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add receive shipments action to table (2.50 KB, patch)
2024-11-11 15:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Restore new basket/contract buttons (1.56 KB, patch)
2024-11-11 15:01 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add dropdown button component (4.48 KB, patch)
2024-11-11 15:01 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Reset basket view (21.16 KB, patch)
2024-11-11 15:01 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary navigation check (1.61 KB, patch)
2024-11-15 16:15 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (31.76 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.57 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (11.38 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.28 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.83 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (46.41 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (5.06 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.79 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.54 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.80 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.04 KB, patch)
2024-12-04 09:31 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.48 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (3.02 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.14 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.03 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (4.26 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.02 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.32 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.53 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.17 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.18 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.32 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (16.27 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (943 bytes, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.91 KB, patch)
2024-12-04 09:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.41 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (5.50 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add receive shipments action to table (2.50 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Restore new basket/contract buttons (3.15 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add dropdown button component (3.48 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Reset basket view (21.16 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary navigation check (1.61 KB, patch)
2024-12-04 09:33 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 37301 (6.84 KB, patch)
2024-12-04 10:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix tax rate display (1.55 KB, patch)
2024-12-04 10:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add callbacks to Toolbarbutton (2.49 KB, patch)
2024-12-04 10:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix acqui-home.pl breadcrumbs (1.76 KB, patch)
2024-12-04 10:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions for additional fields (991 bytes, patch)
2024-12-04 10:03 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix programmatic navigation id attribute (5.43 KB, patch)
2024-12-04 10:55 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary test (151.25 KB, patch)
2024-12-04 11:20 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Delete old code (53.48 KB, patch)
2024-12-04 11:27 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix navigation links (2.37 KB, patch)
2024-12-05 13:21 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Prevent vendor contacts from duplicating (3.94 KB, patch)
2024-12-05 13:58 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Check for baskets for receive shipments button (1.13 KB, patch)
2024-12-05 14:41 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Check for invoices before allowing vendor deletion (3.77 KB, patch)
2024-12-05 14:41 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Only allow one primary contact (6.84 KB, patch)
2024-12-05 15:29 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary buttons (2.28 KB, patch)
2024-12-05 15:32 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Apply format_date to vendor contracts (3.45 KB, patch)
2024-12-06 15:41 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Change how interface details are displayed (1.37 KB, patch)
2024-12-19 17:00 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Apply format_date to vendor contracts (3.48 KB, patch)
2024-12-19 18:16 UTC, PTFS Europe Sandboxes
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Template fixes (1.19 KB, patch)
2025-01-15 10:43 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (31.19 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.25 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions (10.40 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add authorised values (8.57 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (3.91 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (45.58 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update navigation links (4.86 KB, patch)
2025-02-18 13:46 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.51 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.27 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.53 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.79 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.02 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.46 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (2.86 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.31 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.00 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (4.26 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.08 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.85 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.52 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.16 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.28 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.31 KB, patch)
2025-02-18 13:47 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (4.90 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.86 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix rspack error in VendorShow (932 bytes, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (2.90 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.39 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (5.49 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add receive shipments action to table (2.49 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Restore new basket/contract buttons (3.14 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add dropdown button component (3.47 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Reset basket view (21.23 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary navigation check (1.59 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix tax rate display (1.54 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Add callbacks to Toolbarbutton (2.42 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix acqui-home.pl breadcrumbs (1.74 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix programmatic navigation id attribute (4.35 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary test (151.23 KB, patch)
2025-02-18 13:48 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Delete old code (54.67 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix navigation links (2.25 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Prevent vendor contacts from duplicating (3.86 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Check for baskets for receive shipments button (1.12 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Check for invoices before allowing vendor deletion (3.53 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Only allow one primary contact (6.83 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary buttons (2.25 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Apply format_date to vendor contracts (3.44 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Change how interface details are displayed (1.35 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Template fixes (1.17 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix new vendor form (5.52 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Restore navigation links (5.26 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix delete button (2.06 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix vendor show component (1.13 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Use format_price (1.76 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase acquisitions amd vendor menus (2.47 KB, patch)
2025-02-18 13:49 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Remove unnecessary code from booksellers.pl (8.55 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase for bug 37930 (11.10 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase for bug 38854 (1.00 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Convert vendor menu to Vue island (6.20 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add unit tests (4.60 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix toolbar on VendorShow (5.13 KB, patch)
2025-02-18 13:50 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add a 'Show password' option (1.71 KB, patch)
2025-04-28 09:17 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (bugfix) Show contract of the basket if inactive (1.58 KB, patch)
2025-04-28 10:38 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add vue app and list component (30.65 KB, patch)
2025-04-30 20:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add the component to display a vendor (23.34 KB, patch)
2025-04-30 20:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add permissions (10.13 KB, patch)
2025-04-30 20:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add authorised values (6.88 KB, patch)
2025-04-30 20:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Rename routes to acquisitions (4.04 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add the form to add a new vendor (45.71 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Update navigation links (4.98 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Allow external breadcrumbs (2.64 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add a contracts table to the vendor show component (8.40 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Remove drawCallback from contracts table (4.65 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add permissions to action buttons (3.92 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions (2.15 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add cypress tests (12.59 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Adjust vendor update logic (2.99 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix vendors search bar (6.46 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Update links to vendor module (49.15 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Rebase for Button component change (4.40 KB, patch)
2025-04-30 20:59 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add basket page (17.18 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Allow embedding for baskets table (15.98 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Point basket action buttons to correct pages (1.65 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix null basket names (2.28 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Respect AcqViewBaskets syspref (3.41 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add delete contact button (1.43 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add menu components for vendor and acquisitions menus (5.02 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Implement side menus in vendors app (4.90 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Allow searching by alias in the vendors table (3.02 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Form fixes (2.52 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Rebase for bug 38390 (5.58 KB, patch)
2025-04-30 21:00 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add receive shipments action to table (2.61 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Restore new basket/contract buttons (3.27 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add dropdown button component (3.60 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Reset basket view (21.39 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary navigation check (1.72 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix tax rate display (1.67 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add callbacks to Toolbarbutton (2.54 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix acqui-home.pl breadcrumbs (1.89 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix programmatic navigation id attribute (4.47 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Remove unneccessary test (1.89 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Delete old code (54.61 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix navigation links (2.38 KB, patch)
2025-04-30 21:01 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Prevent vendor contacts from duplicating (3.98 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Check for baskets for receive shipments button (1.24 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Check for invoices before allowing vendor deletion (3.66 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Only allow one primary contact (6.95 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Remove unnecessary buttons (2.38 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Apply format_date to vendor contracts (3.57 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Change how interface details are displayed (1.47 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Template fixes (1.30 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix new vendor form (5.65 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Restore navigation links (5.39 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix delete button (2.19 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix vendor show component (1.25 KB, patch)
2025-04-30 21:02 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Use format_price (1.88 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase acquisitions amd vendor menus (2.60 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Remove unnecessary code from booksellers.pl (8.71 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase for bug 37930 (11.87 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase for bug 38854 (1.13 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Convert vendor menu to Vue island (6.38 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add unit tests (4.73 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix toolbar on VendorShow (5.23 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Update routes to include 'acquisition' (52.64 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Redirect Home component to List (1.16 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Move island hydration to container wrapper (2.63 KB, patch)
2025-04-30 21:03 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix issues in Show component (9.71 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Fix issues on vendor edit page (8.61 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Adjust mapping for the manual (2.12 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) UI fixes (7.63 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Handle form parameters in the browser (2.33 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Rebase on main (5.77 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Convert delete modal to use the API client (1.45 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add a config endpoint for permissions (7.14 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Email type checking and UI fixes (3.62 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Stop breadcrumbs from keeping filters (5.24 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add new apache redirect (1.28 KB, patch)
2025-04-30 21:04 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add a 'Show password' option (1.83 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Improve show/hide password icon (3.20 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add a Link component (7.16 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add missing filter to main-container.inc (1.57 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Remove extra sidebar_menu class (2.26 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Remove op=do_search from the 'Invoices' link (1.25 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Code improvement (2.38 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix links to the manual (1.76 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix wrong use of scalar in the REST API controller (1.99 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add tests for embed contacts,interfaces,aliases (7.86 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Move syspref values to config endpoint (5.57 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix LeftMenu (1.66 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Embed baskets+count and subscriptions+count (11.53 KB, patch)
2025-04-30 21:05 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Migrate to_api logic to GET endpoint (3.76 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Translation fix (857 bytes, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add protection logic to the DELETE endpoint (2.24 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (QA follow-up) Add default settings to vendor store (967 bytes, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix vendor deletion tests (1.51 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Add tests for DELETE vendors/{vendor_id} (2.42 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix cypress tests - do not intercept /config (3.86 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix cypress tests - add _count to the mock object (1.46 KB, patch)
2025-04-30 21:06 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Fix xt/api.t (735 bytes, patch)
2025-05-05 07:50 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Embed invoices_count - fix patron deletion from show view (3.02 KB, patch)
2025-05-05 08:34 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: Restore ordering on the contracts table (1.79 KB, patch)
2025-05-05 09:16 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (follow-up) Fix API spec terminology (1.06 KB, patch)
2025-05-06 10:16 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: (follow-up) Make vendor table columns configurable (3.14 KB, patch)
2025-05-06 10:16 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Restore ordering on the contracts table (8.06 KB, patch)
2025-05-07 21:16 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 38010: (follow-up) Remove DT reference (1.08 KB, patch)
2025-05-13 10:11 UTC, Matt Blenkinsop
Details | Diff | Splinter Review
Bug 38010: Fix cypress tests by mocking more API responses (1.60 KB, patch)
2025-05-14 11:38 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Matt Blenkinsop 2024-09-26 13:15:48 UTC

    
Comment 1 David Cook 2024-10-30 04:15:38 UTC
Ohhh this would be cool. Might also be easier to find people to test this one. I don't have anyone using Preservation or ERM, but Acquisitions... for sure.
Comment 2 Matt Blenkinsop 2024-11-07 11:48:27 UTC
Created attachment 174119 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 3 Matt Blenkinsop 2024-11-07 11:48:30 UTC
Created attachment 174120 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 4 Matt Blenkinsop 2024-11-07 11:48:32 UTC
Created attachment 174121 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 5 Matt Blenkinsop 2024-11-07 11:48:34 UTC
Created attachment 174122 [details] [review]
Bug 38010: Add authorised values
Comment 6 Matt Blenkinsop 2024-11-07 11:48:36 UTC
Created attachment 174123 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 7 Matt Blenkinsop 2024-11-07 11:48:39 UTC
Created attachment 174124 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 8 Matt Blenkinsop 2024-11-07 11:48:41 UTC
Created attachment 174125 [details] [review]
Bug 38010: Update navigation links
Comment 9 Matt Blenkinsop 2024-11-07 11:48:43 UTC
Created attachment 174126 [details] [review]
Bug 38010: Add a nav link button component
Comment 10 Matt Blenkinsop 2024-11-07 11:48:46 UTC
Created attachment 174128 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 11 Matt Blenkinsop 2024-11-07 11:48:48 UTC
Created attachment 174129 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 12 Matt Blenkinsop 2024-11-07 11:48:51 UTC
Created attachment 174130 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 13 Matt Blenkinsop 2024-11-07 11:48:53 UTC
Created attachment 174131 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 14 Matt Blenkinsop 2024-11-07 11:48:55 UTC
Created attachment 174132 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 15 Matt Blenkinsop 2024-11-07 11:48:58 UTC
Created attachment 174133 [details] [review]
Bug 38010: Add cypress tests
Comment 16 Matt Blenkinsop 2024-11-07 11:49:00 UTC
Created attachment 174134 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 17 Matt Blenkinsop 2024-11-07 11:49:03 UTC
Created attachment 174135 [details] [review]
Bug 38010: Fix vendors search bar
Comment 18 Matt Blenkinsop 2024-11-07 11:49:07 UTC
Created attachment 174136 [details] [review]
Bug 38010: Update links to vendor module
Comment 19 Matt Blenkinsop 2024-11-07 11:49:09 UTC
Created attachment 174137 [details] [review]
Bug 38010: Rebase for Button component change
Comment 20 Matt Blenkinsop 2024-11-07 11:49:12 UTC
Created attachment 174138 [details] [review]
Bug 38010: Add basket page
Comment 21 Matt Blenkinsop 2024-11-07 11:49:15 UTC
Created attachment 174139 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 22 Matt Blenkinsop 2024-11-07 11:49:18 UTC
Created attachment 174140 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 23 Matt Blenkinsop 2024-11-07 11:49:20 UTC
Created attachment 174141 [details] [review]
Bug 38010: Fix null basket names
Comment 24 Matt Blenkinsop 2024-11-07 11:49:23 UTC
Created attachment 174142 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 25 Matt Blenkinsop 2024-11-07 11:49:26 UTC
Created attachment 174143 [details] [review]
Bug 38010: Add delete contact button
Comment 26 Matt Blenkinsop 2024-11-07 11:49:28 UTC
Created attachment 174144 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 27 Matt Blenkinsop 2024-11-07 11:49:30 UTC
Created attachment 174145 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 28 Matt Blenkinsop 2024-11-07 11:49:33 UTC
Created attachment 174146 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 29 Matt Blenkinsop 2024-11-07 11:49:35 UTC
Created attachment 174147 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 30 Matt Blenkinsop 2024-11-07 11:49:38 UTC
Created attachment 174148 [details] [review]
Bug 38010: Form fixes
Comment 31 Matt Blenkinsop 2024-11-07 11:56:01 UTC
Created attachment 174151 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 32 Matt Blenkinsop 2024-11-07 11:56:04 UTC
Created attachment 174152 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 33 Matt Blenkinsop 2024-11-07 11:56:06 UTC
Created attachment 174153 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 34 Matt Blenkinsop 2024-11-07 11:56:09 UTC
Created attachment 174154 [details] [review]
Bug 38010: Add authorised values
Comment 35 Matt Blenkinsop 2024-11-07 11:56:11 UTC
Created attachment 174155 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 36 Matt Blenkinsop 2024-11-07 11:56:14 UTC
Created attachment 174156 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 37 Matt Blenkinsop 2024-11-07 11:56:16 UTC
Created attachment 174157 [details] [review]
Bug 38010: Update navigation links
Comment 38 Matt Blenkinsop 2024-11-07 11:56:19 UTC
Created attachment 174158 [details] [review]
Bug 38010: Add a nav link button component
Comment 39 Matt Blenkinsop 2024-11-07 11:56:22 UTC
Created attachment 174159 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 40 Matt Blenkinsop 2024-11-07 11:56:25 UTC
Created attachment 174160 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 41 Matt Blenkinsop 2024-11-07 11:56:27 UTC
Created attachment 174161 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 42 Matt Blenkinsop 2024-11-07 11:56:30 UTC
Created attachment 174162 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 43 Matt Blenkinsop 2024-11-07 11:56:33 UTC
Created attachment 174163 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 44 Matt Blenkinsop 2024-11-07 11:56:35 UTC
Created attachment 174164 [details] [review]
Bug 38010: Add cypress tests
Comment 45 Matt Blenkinsop 2024-11-07 11:56:38 UTC
Created attachment 174165 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 46 Matt Blenkinsop 2024-11-07 11:56:41 UTC
Created attachment 174166 [details] [review]
Bug 38010: Fix vendors search bar
Comment 47 Matt Blenkinsop 2024-11-07 11:56:43 UTC
Created attachment 174167 [details] [review]
Bug 38010: Update links to vendor module
Comment 48 Matt Blenkinsop 2024-11-07 11:56:46 UTC
Created attachment 174168 [details] [review]
Bug 38010: Rebase for Button component change
Comment 49 Matt Blenkinsop 2024-11-07 11:56:49 UTC
Created attachment 174169 [details] [review]
Bug 38010: Add basket page
Comment 50 Matt Blenkinsop 2024-11-07 11:56:51 UTC
Created attachment 174170 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 51 Matt Blenkinsop 2024-11-07 11:56:54 UTC
Created attachment 174171 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 52 Matt Blenkinsop 2024-11-07 11:56:57 UTC
Created attachment 174172 [details] [review]
Bug 38010: Fix null basket names
Comment 53 Matt Blenkinsop 2024-11-07 11:56:59 UTC
Created attachment 174173 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 54 Matt Blenkinsop 2024-11-07 11:57:02 UTC
Created attachment 174174 [details] [review]
Bug 38010: Add delete contact button
Comment 55 Matt Blenkinsop 2024-11-07 11:57:05 UTC
Created attachment 174175 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 56 Matt Blenkinsop 2024-11-07 11:57:08 UTC
Created attachment 174176 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 57 Matt Blenkinsop 2024-11-07 11:57:10 UTC
Created attachment 174177 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 58 Matt Blenkinsop 2024-11-07 11:57:13 UTC
Created attachment 174178 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 59 Matt Blenkinsop 2024-11-07 11:57:16 UTC
Created attachment 174179 [details] [review]
Bug 38010: Form fixes
Comment 60 Matt Blenkinsop 2024-11-07 13:26:33 UTC
Created attachment 174194 [details] [review]
Bug 38010: Rebase for bug 38390
Comment 61 Matt Blenkinsop 2024-11-07 13:32:36 UTC
Created attachment 174196 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 62 Matt Blenkinsop 2024-11-07 13:32:38 UTC
Created attachment 174197 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 63 Matt Blenkinsop 2024-11-07 13:32:41 UTC
Created attachment 174198 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 64 Matt Blenkinsop 2024-11-07 13:32:44 UTC
Created attachment 174199 [details] [review]
Bug 38010: Add authorised values
Comment 65 Matt Blenkinsop 2024-11-07 13:32:47 UTC
Created attachment 174200 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 66 Matt Blenkinsop 2024-11-07 13:32:50 UTC
Created attachment 174201 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 67 Matt Blenkinsop 2024-11-07 13:32:53 UTC
Created attachment 174202 [details] [review]
Bug 38010: Update navigation links
Comment 68 Matt Blenkinsop 2024-11-07 13:32:56 UTC
Created attachment 174203 [details] [review]
Bug 38010: Add a nav link button component
Comment 69 Matt Blenkinsop 2024-11-07 13:32:59 UTC
Created attachment 174204 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 70 Matt Blenkinsop 2024-11-07 13:33:02 UTC
Created attachment 174205 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 71 Matt Blenkinsop 2024-11-07 13:33:04 UTC
Created attachment 174206 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 72 Matt Blenkinsop 2024-11-07 13:33:07 UTC
Created attachment 174207 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 73 Matt Blenkinsop 2024-11-07 13:33:10 UTC
Created attachment 174208 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 74 Matt Blenkinsop 2024-11-07 13:33:13 UTC
Created attachment 174209 [details] [review]
Bug 38010: Add cypress tests
Comment 75 Matt Blenkinsop 2024-11-07 13:33:16 UTC
Created attachment 174210 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 76 Matt Blenkinsop 2024-11-07 13:33:19 UTC
Created attachment 174211 [details] [review]
Bug 38010: Fix vendors search bar
Comment 77 Matt Blenkinsop 2024-11-07 13:33:22 UTC
Created attachment 174212 [details] [review]
Bug 38010: Update links to vendor module
Comment 78 Matt Blenkinsop 2024-11-07 13:33:25 UTC
Created attachment 174213 [details] [review]
Bug 38010: Rebase for Button component change
Comment 79 Matt Blenkinsop 2024-11-07 13:33:27 UTC
Created attachment 174214 [details] [review]
Bug 38010: Add basket page
Comment 80 Matt Blenkinsop 2024-11-07 13:33:30 UTC
Created attachment 174215 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 81 Matt Blenkinsop 2024-11-07 13:33:33 UTC
Created attachment 174216 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 82 Matt Blenkinsop 2024-11-07 13:33:36 UTC
Created attachment 174217 [details] [review]
Bug 38010: Fix null basket names
Comment 83 Matt Blenkinsop 2024-11-07 13:33:39 UTC
Created attachment 174218 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 84 Matt Blenkinsop 2024-11-07 13:33:42 UTC
Created attachment 174220 [details] [review]
Bug 38010: Add delete contact button
Comment 85 Matt Blenkinsop 2024-11-07 13:33:45 UTC
Created attachment 174221 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 86 Matt Blenkinsop 2024-11-07 13:33:48 UTC
Created attachment 174222 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 87 Matt Blenkinsop 2024-11-07 13:33:50 UTC
Created attachment 174223 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 88 Matt Blenkinsop 2024-11-07 13:33:53 UTC
Created attachment 174224 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 89 Matt Blenkinsop 2024-11-07 13:33:56 UTC
Created attachment 174225 [details] [review]
Bug 38010: Form fixes
Comment 90 Matt Blenkinsop 2024-11-07 13:33:59 UTC
Created attachment 174226 [details] [review]
Bug 38010: Rebase for bug 38390
Comment 91 Matt Blenkinsop 2024-11-07 15:24:34 UTC
Created attachment 174248 [details] [review]
Bug 38010: Add recevie shipments action to table
Comment 92 Matt Blenkinsop 2024-11-08 09:58:08 UTC
Created attachment 174266 [details] [review]
Bug 38010: Add receive shipments action to table
Comment 93 Matt Blenkinsop 2024-11-08 09:58:11 UTC
Created attachment 174267 [details] [review]
Bug 38010: Restore new basket/contract buttons
Comment 94 Matt Blenkinsop 2024-11-08 13:09:25 UTC
Created attachment 174269 [details] [review]
Bug 38010: Add dropdown button component
Comment 95 Matt Blenkinsop 2024-11-08 13:09:28 UTC
Created attachment 174270 [details] [review]
Bug 38010: Reset basket view
Comment 96 Matt Blenkinsop 2024-11-08 14:03:30 UTC
Created attachment 174273 [details] [review]
Bug 38010: Reset basket view
Comment 97 Matt Blenkinsop 2024-11-11 14:59:20 UTC
Created attachment 174327 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 98 Matt Blenkinsop 2024-11-11 14:59:23 UTC
Created attachment 174328 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 99 Matt Blenkinsop 2024-11-11 14:59:26 UTC
Created attachment 174329 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 100 Matt Blenkinsop 2024-11-11 14:59:29 UTC
Created attachment 174330 [details] [review]
Bug 38010: Add authorised values
Comment 101 Matt Blenkinsop 2024-11-11 14:59:32 UTC
Created attachment 174331 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 102 Matt Blenkinsop 2024-11-11 14:59:36 UTC
Created attachment 174332 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 103 Matt Blenkinsop 2024-11-11 14:59:39 UTC
Created attachment 174333 [details] [review]
Bug 38010: Update navigation links
Comment 104 Matt Blenkinsop 2024-11-11 14:59:42 UTC
Created attachment 174334 [details] [review]
Bug 38010: Add a nav link button component
Comment 105 Matt Blenkinsop 2024-11-11 14:59:45 UTC
Created attachment 174335 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 106 Matt Blenkinsop 2024-11-11 14:59:49 UTC
Created attachment 174336 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 107 Matt Blenkinsop 2024-11-11 14:59:52 UTC
Created attachment 174337 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 108 Matt Blenkinsop 2024-11-11 14:59:56 UTC
Created attachment 174338 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 109 Matt Blenkinsop 2024-11-11 14:59:59 UTC
Created attachment 174339 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 110 Matt Blenkinsop 2024-11-11 15:00:02 UTC
Created attachment 174340 [details] [review]
Bug 38010: Add cypress tests
Comment 111 Matt Blenkinsop 2024-11-11 15:00:06 UTC
Created attachment 174341 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 112 Matt Blenkinsop 2024-11-11 15:00:09 UTC
Created attachment 174342 [details] [review]
Bug 38010: Fix vendors search bar
Comment 113 Matt Blenkinsop 2024-11-11 15:00:12 UTC
Created attachment 174343 [details] [review]
Bug 38010: Update links to vendor module
Comment 114 Matt Blenkinsop 2024-11-11 15:00:16 UTC
Created attachment 174344 [details] [review]
Bug 38010: Rebase for Button component change
Comment 115 Matt Blenkinsop 2024-11-11 15:00:19 UTC
Created attachment 174345 [details] [review]
Bug 38010: Add basket page
Comment 116 Matt Blenkinsop 2024-11-11 15:00:22 UTC
Created attachment 174346 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 117 Matt Blenkinsop 2024-11-11 15:00:26 UTC
Created attachment 174347 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 118 Matt Blenkinsop 2024-11-11 15:00:29 UTC
Created attachment 174348 [details] [review]
Bug 38010: Fix null basket names
Comment 119 Matt Blenkinsop 2024-11-11 15:00:32 UTC
Created attachment 174349 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 120 Matt Blenkinsop 2024-11-11 15:00:36 UTC
Created attachment 174350 [details] [review]
Bug 38010: Add delete contact button
Comment 121 Matt Blenkinsop 2024-11-11 15:00:39 UTC
Created attachment 174351 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 122 Matt Blenkinsop 2024-11-11 15:00:42 UTC
Created attachment 174352 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 123 Matt Blenkinsop 2024-11-11 15:00:46 UTC
Created attachment 174353 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 124 Matt Blenkinsop 2024-11-11 15:00:49 UTC
Created attachment 174354 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 125 Matt Blenkinsop 2024-11-11 15:00:52 UTC
Created attachment 174355 [details] [review]
Bug 38010: Form fixes
Comment 126 Matt Blenkinsop 2024-11-11 15:00:55 UTC
Created attachment 174356 [details] [review]
Bug 38010: Rebase for bug 38390
Comment 127 Matt Blenkinsop 2024-11-11 15:00:59 UTC
Created attachment 174357 [details] [review]
Bug 38010: Add receive shipments action to table
Comment 128 Matt Blenkinsop 2024-11-11 15:01:02 UTC
Created attachment 174358 [details] [review]
Bug 38010: Restore new basket/contract buttons
Comment 129 Matt Blenkinsop 2024-11-11 15:01:05 UTC
Created attachment 174359 [details] [review]
Bug 38010: Add dropdown button component
Comment 130 Matt Blenkinsop 2024-11-11 15:01:09 UTC
Created attachment 174360 [details] [review]
Bug 38010: Reset basket view
Comment 131 Matt Blenkinsop 2024-11-15 16:15:10 UTC
Created attachment 174603 [details] [review]
Bug 38010: Remove unnecessary navigation check
Comment 132 Owen Leonard 2024-12-02 18:28:25 UTC
I get a 404 error when I try to search for a vendor or follow a link to a vendor.
Comment 133 Caroline Cyr La Rose 2024-12-03 15:15:22 UTC
Hi Matt, not sure if this is just a thing with the sandbox or not, but when I go to edit vendor Askews & Holt in the sandbox, the library group says NaN, 1, NaN

https://staff-vendors.sandboxes.ptfs-europe.co.uk/cgi-bin/koha/vendors/edit/4

To get there :
1. Go to the sandbox https://staff-vendors.sandboxes.ptfs-europe.co.uk/cgi-bin/koha/vendors
2. Click Askews & Holt (#4)
3. Click Edit vendor
--> Library group visiblity: NaN, 1, NaN

Also, is it normal that when we are in the vendors, the left side menu only lists vendors?
Comment 134 Caroline Cyr La Rose 2024-12-03 15:20:27 UTC
(In reply to Caroline Cyr La Rose from comment #133)
> 
> Also, is it normal that when we are in the vendors, the left side menu only
> lists vendors?

To get there:
1. Go to the sandbox https://staff-vendors.sandboxes.ptfs-europe.co.uk/cgi-bin/koha/vendors
2. Click Askews & Holt (#4)
3. Click Edit vendor
--> The left side menu only contains Vendors
Comment 135 Matt Blenkinsop 2024-12-04 09:31:14 UTC
Created attachment 175098 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 136 Matt Blenkinsop 2024-12-04 09:31:17 UTC
Created attachment 175099 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 137 Matt Blenkinsop 2024-12-04 09:31:20 UTC
Created attachment 175100 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 138 Matt Blenkinsop 2024-12-04 09:31:24 UTC
Created attachment 175101 [details] [review]
Bug 38010: Add authorised values
Comment 139 Matt Blenkinsop 2024-12-04 09:31:27 UTC
Created attachment 175102 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 140 Matt Blenkinsop 2024-12-04 09:31:33 UTC
Created attachment 175103 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 141 Matt Blenkinsop 2024-12-04 09:31:37 UTC
Created attachment 175104 [details] [review]
Bug 38010: Update navigation links
Comment 142 Matt Blenkinsop 2024-12-04 09:31:42 UTC
Created attachment 175105 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 143 Matt Blenkinsop 2024-12-04 09:31:46 UTC
Created attachment 175106 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 144 Matt Blenkinsop 2024-12-04 09:31:49 UTC
Created attachment 175107 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 145 Matt Blenkinsop 2024-12-04 09:31:53 UTC
Created attachment 175108 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 146 Matt Blenkinsop 2024-12-04 09:31:58 UTC
Created attachment 175109 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 147 Matt Blenkinsop 2024-12-04 09:32:03 UTC
Created attachment 175110 [details] [review]
Bug 38010: Add cypress tests
Comment 148 Matt Blenkinsop 2024-12-04 09:32:08 UTC
Created attachment 175111 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 149 Matt Blenkinsop 2024-12-04 09:32:12 UTC
Created attachment 175112 [details] [review]
Bug 38010: Fix vendors search bar
Comment 150 Matt Blenkinsop 2024-12-04 09:32:16 UTC
Created attachment 175113 [details] [review]
Bug 38010: Update links to vendor module
Comment 151 Matt Blenkinsop 2024-12-04 09:32:19 UTC
Created attachment 175114 [details] [review]
Bug 38010: Rebase for Button component change
Comment 152 Matt Blenkinsop 2024-12-04 09:32:22 UTC
Created attachment 175115 [details] [review]
Bug 38010: Add basket page
Comment 153 Matt Blenkinsop 2024-12-04 09:32:26 UTC
Created attachment 175116 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 154 Matt Blenkinsop 2024-12-04 09:32:29 UTC
Created attachment 175117 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 155 Matt Blenkinsop 2024-12-04 09:32:33 UTC
Created attachment 175118 [details] [review]
Bug 38010: Fix null basket names
Comment 156 Matt Blenkinsop 2024-12-04 09:32:36 UTC
Created attachment 175119 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 157 Matt Blenkinsop 2024-12-04 09:32:39 UTC
Created attachment 175120 [details] [review]
Bug 38010: Add delete contact button
Comment 158 Matt Blenkinsop 2024-12-04 09:32:45 UTC
Created attachment 175121 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 159 Matt Blenkinsop 2024-12-04 09:32:50 UTC
Created attachment 175122 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 160 Matt Blenkinsop 2024-12-04 09:32:54 UTC
Created attachment 175123 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 161 Matt Blenkinsop 2024-12-04 09:32:58 UTC
Created attachment 175124 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 162 Matt Blenkinsop 2024-12-04 09:33:02 UTC
Created attachment 175125 [details] [review]
Bug 38010: Form fixes
Comment 163 Matt Blenkinsop 2024-12-04 09:33:05 UTC
Created attachment 175126 [details] [review]
Bug 38010: Rebase for bug 38390
Comment 164 Matt Blenkinsop 2024-12-04 09:33:09 UTC
Created attachment 175127 [details] [review]
Bug 38010: Add receive shipments action to table
Comment 165 Matt Blenkinsop 2024-12-04 09:33:12 UTC
Created attachment 175128 [details] [review]
Bug 38010: Restore new basket/contract buttons
Comment 166 Matt Blenkinsop 2024-12-04 09:33:16 UTC
Created attachment 175129 [details] [review]
Bug 38010: Add dropdown button component
Comment 167 Matt Blenkinsop 2024-12-04 09:33:19 UTC
Created attachment 175130 [details] [review]
Bug 38010: Reset basket view
Comment 168 Matt Blenkinsop 2024-12-04 09:33:22 UTC
Created attachment 175131 [details] [review]
Bug 38010: Remove unnecessary navigation check
Comment 169 Matt Blenkinsop 2024-12-04 10:03:35 UTC
Created attachment 175133 [details] [review]
Bug 38010: Rebase for bug 37301
Comment 170 Matt Blenkinsop 2024-12-04 10:03:38 UTC
Created attachment 175134 [details] [review]
Bug 38010: Fix tax rate display
Comment 171 Matt Blenkinsop 2024-12-04 10:03:42 UTC
Created attachment 175135 [details] [review]
Bug 38010: Add callbacks to Toolbarbutton
Comment 172 Matt Blenkinsop 2024-12-04 10:03:48 UTC
Created attachment 175136 [details] [review]
Bug 38010: Fix acqui-home.pl breadcrumbs
Comment 173 Matt Blenkinsop 2024-12-04 10:03:54 UTC
Created attachment 175137 [details] [review]
Bug 38010: Add permissions for additional fields

(cherry picked from commit 0387e998a49b91a57c798504c28a097a011a9d77)
Comment 174 Matt Blenkinsop 2024-12-04 10:55:14 UTC
Created attachment 175167 [details] [review]
Bug 38010: Fix programmatic navigation id attribute
Comment 175 Matt Blenkinsop 2024-12-04 10:58:18 UTC
(In reply to Owen Leonard from comment #132)
> I get a 404 error when I try to search for a vendor or follow a link to a
> vendor.

This sounds like an apache issue - you need to run reset_all after applying to get the apache redirect to activate
Comment 176 Matt Blenkinsop 2024-12-04 10:59:30 UTC
(In reply to Caroline Cyr La Rose from comment #133)
> Hi Matt, not sure if this is just a thing with the sandbox or not, but when
> I go to edit vendor Askews & Holt in the sandbox, the library group says
> NaN, 1, NaN

This is sandbox specific, that includes lots of further enhancements that aren't part of this bug

> 
> Also, is it normal that when we are in the vendors, the left side menu only
> lists vendors?

I've removed this so that there is no menu which is what we currently have in main
Comment 177 Katrin Fischer 2024-12-04 11:00:15 UTC
(In reply to Matt Blenkinsop from comment #175)
> (In reply to Owen Leonard from comment #132)
> > I get a 404 error when I try to search for a vendor or follow a link to a
> > vendor.
> 
> This sounds like an apache issue - you need to run reset_all after applying
> to get the apache redirect to activate

Wondering (in general) how we handle this for updating installations - I believe we can't just add to the Apache files, but maybe I am mistaken. Hint on about?
Comment 178 Matt Blenkinsop 2024-12-04 11:20:40 UTC
Created attachment 175168 [details] [review]
Bug 38010: Remove unnecessary test
Comment 179 Matt Blenkinsop 2024-12-04 11:27:46 UTC
Created attachment 175170 [details] [review]
Bug 38010: Delete old code
Comment 180 Michaela Sieber 2024-12-05 13:10:08 UTC
I started testing and found two bugs which you can recreate in this sandbox :

1.) Error "Something went wrong: Error: Expected integer - got string."

Go to acq module
search for vendors
Go to record 1 = "My Vendor"
Click on Baskets on the left
Click on Button "Edit vendor" on the top. This leads you to the following URL:
https://staff-vendorsvue.sandboxes.ptfs-europe.co.uk/cgi-bin/koha/vendors/1&op=enter

--> Error: "Something went wrong: Error: Expected integer - got string."

2.) Error 404

Go to acq module
search for vendors
Go to record 1 = "My Vendor"
Click on Baskets on the left 
Go to the existing basket "My Basket" (basketno=1)
Click on the link to the vendor "My Vendor" on the top. This leads you to the following URL:
https://staff-vendorsvue.sandboxes.ptfs-europe.co.uk/cgi-bin/koha/acqui/cgi-bin/koha/vendors/1

--> Error 404
Comment 181 Matt Blenkinsop 2024-12-05 13:21:25 UTC
Created attachment 175240 [details] [review]
Bug 38010: Fix navigation links
Comment 182 Matt Blenkinsop 2024-12-05 13:58:52 UTC
Created attachment 175241 [details] [review]
Bug 38010: Prevent vendor contacts from duplicating
Comment 183 Michaela Sieber 2024-12-05 14:12:36 UTC
Things I discovered that are new / different behavior as before:

- New vendors are inactive (radio button default is inactive, before it was active)

- There is a new Button "New basket", this opens a new browser tab (did not exist before)

- The Button "Receive shipment" exists even if there is no basket for the vendor (before the botton was only there if there was at least one basket). This means you can enter an invoice, for example with invoice adjustments. That is ok, but then it should not be possible to delete the vendor. If you delete the vendor the invoice is deleted as well. This means your budget is not correct any more! 
If there are invoices in the database it should definitly not be possible to delete a vendor.
Comment 184 Matt Blenkinsop 2024-12-05 14:41:31 UTC
Created attachment 175242 [details] [review]
Bug 38010: Check for baskets for receive shipments button
Comment 185 Matt Blenkinsop 2024-12-05 14:41:35 UTC
Created attachment 175243 [details] [review]
Bug 38010: Check for invoices before allowing vendor deletion
Comment 186 Matt Blenkinsop 2024-12-05 15:29:59 UTC
Created attachment 175248 [details] [review]
Bug 38010: Only allow one primary contact
Comment 187 Matt Blenkinsop 2024-12-05 15:32:44 UTC
Created attachment 175249 [details] [review]
Bug 38010: Remove unnecessary buttons
Comment 188 Matt Blenkinsop 2024-12-06 15:41:22 UTC
Created attachment 175266 [details] [review]
Bug 38010: Apply format_date to vendor contracts
Comment 189 Matt Blenkinsop 2024-12-19 17:00:58 UTC
Created attachment 175791 [details] [review]
Bug 38010: Change how interface details are displayed
Comment 190 PTFS Europe Sandboxes 2024-12-19 18:16:31 UTC
Created attachment 175799 [details] [review]
Bug 38010: Apply format_date to vendor contracts

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Comment 191 Michaela Sieber 2024-12-19 18:20:52 UTC
Well done Matt! Ready for QA
Comment 192 Ola Andersson 2024-12-20 11:41:16 UTC
I was asked to leave this comment confirming performance issues with the acquisition module searching for vendors with thousands of baskets. Hopefully it will be resolved with this work.

"We have a 25 second waiting time when we search for a vendor in the acquisition module that had 5000+ baskets."
Comment 193 Katrin Fischer 2024-12-20 12:17:38 UTC
(In reply to Ola Andersson from comment #192)
> I was asked to leave this comment confirming performance issues with the
> acquisition module searching for vendors with thousands of baskets.
> Hopefully it will be resolved with this work.
> 
> "We have a 25 second waiting time when we search for a vendor in the
> acquisition module that had 5000+ baskets."

I thought it might make a good stress test for the new vendor search :)
Comment 194 Jonathan Druart 2025-01-02 16:35:11 UTC
Had a quick look at the UI, some notes:
1. "Save" => "Submit"
2. No more sticky toolbar
3.
 a. "Add vendor" => "New vendor"
 b. It's not h2 instead of h1
4. Alias
 a. can be added empty (was not possible before)
 b. "Remove" does not remove the correct alias
5. http://localhost:8081/cgi-bin/koha/acqui/booksellers.pl?booksellerid=2
"Edit" is http://localhost:8081/cgi-bin/koha/vendors/2 ("show" view)
6. Empty contact can be added (was not possible before)
7. "Delete contact" and "Remove this interface" is not consistent (in wording, in position, and in styling)
8. Discount and delivery time have type="number"
https://wiki.koha-community.org/wiki/Coding_Guidelines#ACC2:_Input_type_.22number.22_should_be_avoided
9. Delete not working
http://localhost:8081/cgi-bin/koha/acqui/booksellers.pl?booksellerid=2
"Delete vendor", yes
=> 404 and vendor is not deleted.
10. Breadcrumb
 a. Small glitch in the breadcrumb (spacing difference before 'Vendors'?)
https://snipboard.io/GQ5XRs.jpg
vs
https://snipboard.io/fubCc5.jpg
 b. http://localhost:8081/cgi-bin/koha/vendors
'Vendors' is a link (http://localhost:8081/cgi-bin/koha/vendors#)
11. Interfaces
 a. On the "show" view, "Type" is displayed when not filled (was not before)
 b. The name of the interface is now in a `legend` tag, was h3 before. I don't think you can use `legend` for something else than `fieldset`.
12. We are missing tests. You should have a look at bug 38461 and bug 38503.
Comment 195 Jonathan Druart 2025-01-02 16:39:14 UTC
13. The vendor search result is really different from what we have currently, and it's missing a lot of info and actions (basket list, "add to basket", "show all baskets", "show active vendors only", the ability to basket or contract for a vendor). Has this been discussed before already?
Comment 196 Jonathan Druart 2025-01-02 16:58:32 UTC
The permissions store is a bit weird as you rely on a global JS variable. How could this be reusable from the ERM module for instance?

It would really have been great to have such concepts/ideas isolated into their own bug instead of lost in this huge set...
Comment 197 Jonathan Druart 2025-01-02 16:58:58 UTC
Should not we move js/vue/components/Vendors to js/vue/components/Acquisitions/Vendors?
Comment 198 Jonathan Druart 2025-01-02 17:03:14 UTC
+            `${(vendor.tax_rate * 100).toFixed(2)}` + "{enter}",

+                    label: `${(gv.option * 100).toFixed(2)}%`,

I don't think this is correct, you should use format_price (from format_price.inc)
Comment 199 Jonathan Druart 2025-01-02 17:04:52 UTC
Wrong paste

koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/AgreementResource.vue:            //AgreementResource specific setup here
koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorResource.vue:            //AgreementResource specific setup here
Comment 200 Jonathan Druart 2025-01-02 17:08:03 UTC
What's the purpose of VendorDetails.vue? I don't see how it could be useful.
Comment 201 Jonathan Druart 2025-01-02 17:12:38 UTC
It seems that this file contains things that are not from this patch set.
koha-tmpl/intranet-tmpl/prog/js/vue/components/Islands/AcquisitionsMenu.vue
Comment 202 Jonathan Druart 2025-01-02 17:13:46 UTC
Code duplication, the acquisition menu is now in koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc and koha-tmpl/intranet-tmpl/prog/js/vue/components/Islands/AcquisitionsMenu.vue
This should be on its own bug really. Replace the left menu everywhere before this and don't duplicate the code.
Comment 203 Jonathan Druart 2025-01-02 17:21:36 UTC
(In reply to Jonathan Druart from comment #194)
> 12. We are missing tests. You should have a look at bug 38461 and bug 38503.

Also missing tests for the pm changes.
Comment 204 Jonathan Druart 2025-01-02 17:26:52 UTC
Koha/Acquisition/Bookseller.pm
+            $interface->password( $interface->plain_text_password );

Do we really want to expose the password? Maybe not something for now, but we should maybe have the "****" trick here with a "show" that will trigger a special route for plain text password (just for discussion as I am reading this code)
Comment 205 Jonathan Druart 2025-01-02 17:30:52 UTC
Koha/Schema/Result/Aqbookseller.pm
+__PACKAGE__->has_many(
+    "aliases",
+    "Koha::Schema::Result::AqbooksellerAlias",

Why?

We already have Koha::Acquisition::Bookseller->aliases, when is this used?
Comment 206 Matt Blenkinsop 2025-01-15 10:43:53 UTC
Created attachment 176547 [details] [review]
Bug 38010: (QA follow-up) Template fixes
Comment 207 Matt Blenkinsop 2025-01-15 10:44:24 UTC
First of a few follow-up patches, this one is needed to make 38899 work. More to follow
Comment 208 Matt Blenkinsop 2025-02-18 13:46:36 UTC
Created attachment 178182 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors
Comment 209 Matt Blenkinsop 2025-02-18 13:46:40 UTC
Created attachment 178183 [details] [review]
Bug 38010: Add the component to display a vendor
Comment 210 Matt Blenkinsop 2025-02-18 13:46:44 UTC
Created attachment 178184 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app
Comment 211 Matt Blenkinsop 2025-02-18 13:46:47 UTC
Created attachment 178185 [details] [review]
Bug 38010: Add authorised values
Comment 212 Matt Blenkinsop 2025-02-18 13:46:51 UTC
Created attachment 178186 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated
Comment 213 Matt Blenkinsop 2025-02-18 13:46:54 UTC
Created attachment 178187 [details] [review]
Bug 38010: Add the form to add a new vendor
Comment 214 Matt Blenkinsop 2025-02-18 13:46:58 UTC
Created attachment 178188 [details] [review]
Bug 38010: Update navigation links
Comment 215 Matt Blenkinsop 2025-02-18 13:47:02 UTC
Created attachment 178189 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application
Comment 216 Matt Blenkinsop 2025-02-18 13:47:05 UTC
Created attachment 178190 [details] [review]
Bug 38010: Add a contracts table to the vendor show component
Comment 217 Matt Blenkinsop 2025-02-18 13:47:09 UTC
Created attachment 178191 [details] [review]
Bug 38010: Remove drawCallback from contracts table
Comment 218 Matt Blenkinsop 2025-02-18 13:47:13 UTC
Created attachment 178192 [details] [review]
Bug 38010: Add permissions to action buttons
Comment 219 Matt Blenkinsop 2025-02-18 13:47:16 UTC
Created attachment 178193 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions
Comment 220 Matt Blenkinsop 2025-02-18 13:47:20 UTC
Created attachment 178194 [details] [review]
Bug 38010: Add cypress tests
Comment 221 Matt Blenkinsop 2025-02-18 13:47:23 UTC
Created attachment 178195 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor
Comment 222 Matt Blenkinsop 2025-02-18 13:47:27 UTC
Created attachment 178196 [details] [review]
Bug 38010: Fix vendors search bar
Comment 223 Matt Blenkinsop 2025-02-18 13:47:31 UTC
Created attachment 178197 [details] [review]
Bug 38010: Update links to vendor module
Comment 224 Matt Blenkinsop 2025-02-18 13:47:34 UTC
Created attachment 178198 [details] [review]
Bug 38010: Rebase for Button component change
Comment 225 Matt Blenkinsop 2025-02-18 13:47:38 UTC
Created attachment 178199 [details] [review]
Bug 38010: Add basket page
Comment 226 Matt Blenkinsop 2025-02-18 13:47:42 UTC
Created attachment 178200 [details] [review]
Bug 38010: Allow embedding for baskets table
Comment 227 Matt Blenkinsop 2025-02-18 13:47:45 UTC
Created attachment 178201 [details] [review]
Bug 38010: Point basket action buttons to correct pages
Comment 228 Matt Blenkinsop 2025-02-18 13:47:49 UTC
Created attachment 178202 [details] [review]
Bug 38010: Fix null basket names
Comment 229 Matt Blenkinsop 2025-02-18 13:47:53 UTC
Created attachment 178203 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref
Comment 230 Matt Blenkinsop 2025-02-18 13:47:56 UTC
Created attachment 178204 [details] [review]
Bug 38010: Add delete contact button
Comment 231 Matt Blenkinsop 2025-02-18 13:48:00 UTC
Created attachment 178205 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus
Comment 232 Matt Blenkinsop 2025-02-18 13:48:04 UTC
Created attachment 178206 [details] [review]
Bug 38010: Implement side menus in vendors app
Comment 233 Matt Blenkinsop 2025-02-18 13:48:07 UTC
Created attachment 178207 [details] [review]
Bug 38010: Fix rspack error in VendorShow
Comment 234 Matt Blenkinsop 2025-02-18 13:48:11 UTC
Created attachment 178208 [details] [review]
Bug 38010: Allow searching by alias in the vendors table
Comment 235 Matt Blenkinsop 2025-02-18 13:48:15 UTC
Created attachment 178209 [details] [review]
Bug 38010: Form fixes
Comment 236 Matt Blenkinsop 2025-02-18 13:48:19 UTC
Created attachment 178210 [details] [review]
Bug 38010: Rebase for bug 38390
Comment 237 Matt Blenkinsop 2025-02-18 13:48:22 UTC
Created attachment 178211 [details] [review]
Bug 38010: Add receive shipments action to table
Comment 238 Matt Blenkinsop 2025-02-18 13:48:26 UTC
Created attachment 178212 [details] [review]
Bug 38010: Restore new basket/contract buttons
Comment 239 Matt Blenkinsop 2025-02-18 13:48:30 UTC
Created attachment 178213 [details] [review]
Bug 38010: Add dropdown button component
Comment 240 Matt Blenkinsop 2025-02-18 13:48:34 UTC
Created attachment 178214 [details] [review]
Bug 38010: Reset basket view
Comment 241 Matt Blenkinsop 2025-02-18 13:48:38 UTC
Created attachment 178215 [details] [review]
Bug 38010: Remove unnecessary navigation check
Comment 242 Matt Blenkinsop 2025-02-18 13:48:42 UTC
Created attachment 178216 [details] [review]
Bug 38010: Fix tax rate display
Comment 243 Matt Blenkinsop 2025-02-18 13:48:45 UTC
Created attachment 178217 [details] [review]
Bug 38010: Add callbacks to Toolbarbutton
Comment 244 Matt Blenkinsop 2025-02-18 13:48:49 UTC
Created attachment 178218 [details] [review]
Bug 38010: Fix acqui-home.pl breadcrumbs
Comment 245 Matt Blenkinsop 2025-02-18 13:48:53 UTC
Created attachment 178219 [details] [review]
Bug 38010: Fix programmatic navigation id attribute
Comment 246 Matt Blenkinsop 2025-02-18 13:48:57 UTC
Created attachment 178220 [details] [review]
Bug 38010: Remove unnecessary test
Comment 247 Matt Blenkinsop 2025-02-18 13:49:00 UTC
Created attachment 178221 [details] [review]
Bug 38010: Delete old code
Comment 248 Matt Blenkinsop 2025-02-18 13:49:04 UTC
Created attachment 178222 [details] [review]
Bug 38010: Fix navigation links
Comment 249 Matt Blenkinsop 2025-02-18 13:49:08 UTC
Created attachment 178223 [details] [review]
Bug 38010: Prevent vendor contacts from duplicating
Comment 250 Matt Blenkinsop 2025-02-18 13:49:12 UTC
Created attachment 178224 [details] [review]
Bug 38010: Check for baskets for receive shipments button
Comment 251 Matt Blenkinsop 2025-02-18 13:49:15 UTC
Created attachment 178225 [details] [review]
Bug 38010: Check for invoices before allowing vendor deletion
Comment 252 Matt Blenkinsop 2025-02-18 13:49:19 UTC
Created attachment 178226 [details] [review]
Bug 38010: Only allow one primary contact
Comment 253 Matt Blenkinsop 2025-02-18 13:49:23 UTC
Created attachment 178227 [details] [review]
Bug 38010: Remove unnecessary buttons
Comment 254 Matt Blenkinsop 2025-02-18 13:49:27 UTC
Created attachment 178228 [details] [review]
Bug 38010: Apply format_date to vendor contracts
Comment 255 Matt Blenkinsop 2025-02-18 13:49:31 UTC
Created attachment 178229 [details] [review]
Bug 38010: Change how interface details are displayed
Comment 256 Matt Blenkinsop 2025-02-18 13:49:35 UTC
Created attachment 178230 [details] [review]
Bug 38010: (QA follow-up) Template fixes
Comment 257 Matt Blenkinsop 2025-02-18 13:49:39 UTC
Created attachment 178231 [details] [review]
Bug 38010: (QA follow-up) Fix new vendor form
Comment 258 Matt Blenkinsop 2025-02-18 13:49:43 UTC
Created attachment 178232 [details] [review]
Bug 38010: (QA follow-up) Restore navigation links

This patch restores any links that previously pointed to booksellers.pl as this script is still in use. All references to supplier.pl remain pointed at the new Vue app
Comment 259 Matt Blenkinsop 2025-02-18 13:49:46 UTC
Created attachment 178233 [details] [review]
Bug 38010: (QA follow-up) Fix delete button
Comment 260 Matt Blenkinsop 2025-02-18 13:49:50 UTC
Created attachment 178234 [details] [review]
Bug 38010: (QA follow-up) Fix vendor show component
Comment 261 Matt Blenkinsop 2025-02-18 13:49:54 UTC
Created attachment 178235 [details] [review]
Bug 38010: (QA follow-up) Use format_price
Comment 262 Matt Blenkinsop 2025-02-18 13:49:58 UTC
Created attachment 178236 [details] [review]
Bug 38010: (QA follow-up) Rebase acquisitions amd vendor menus
Comment 263 Matt Blenkinsop 2025-02-18 13:50:01 UTC
Created attachment 178237 [details] [review]
Bug 38010: (QA follow-up) Remove unnecessary code from booksellers.pl
Comment 264 Matt Blenkinsop 2025-02-18 13:50:05 UTC
Created attachment 178238 [details] [review]
Bug 38010: (QA follow-up) Rebase for bug 37930
Comment 265 Matt Blenkinsop 2025-02-18 13:50:09 UTC
Created attachment 178239 [details] [review]
Bug 38010: (QA follow-up) Rebase for bug 38854
Comment 266 Matt Blenkinsop 2025-02-18 13:50:13 UTC
Created attachment 178240 [details] [review]
Bug 38010: (QA follow-up) Convert vendor menu to Vue island
Comment 267 Matt Blenkinsop 2025-02-18 13:50:17 UTC
Created attachment 178241 [details] [review]
Bug 38010: (QA follow-up) Add unit tests
Comment 268 Matt Blenkinsop 2025-02-18 13:50:21 UTC
Created attachment 178242 [details] [review]
Bug 38010: (QA follow-up) Fix toolbar on VendorShow

The delete button was missing from VendorShow and bug 37301 has been removed as a dependency so some small fixes were required for that
Comment 269 Matt Blenkinsop 2025-02-18 13:53:26 UTC
(In reply to Jonathan Druart from comment #194)
> Had a quick look at the UI, some notes:
> 1. "Save" => "Submit"
> 2. No more sticky toolbar
> 3.
>  a. "Add vendor" => "New vendor"
>  b. It's not h2 instead of h1
> 4. Alias
>  a. can be added empty (was not possible before)
>  b. "Remove" does not remove the correct alias
> 5. http://localhost:8081/cgi-bin/koha/acqui/booksellers.pl?booksellerid=2
> "Edit" is http://localhost:8081/cgi-bin/koha/vendors/2 ("show" view)
> 6. Empty contact can be added (was not possible before)
> 7. "Delete contact" and "Remove this interface" is not consistent (in
> wording, in position, and in styling)
> 8. Discount and delivery time have type="number"
> https://wiki.koha-community.org/wiki/Coding_Guidelines#ACC2:_Input_type_.
> 22number.22_should_be_avoided
> 9. Delete not working
> http://localhost:8081/cgi-bin/koha/acqui/booksellers.pl?booksellerid=2
> "Delete vendor", yes
> => 404 and vendor is not deleted.
> 10. Breadcrumb
>  a. Small glitch in the breadcrumb (spacing difference before 'Vendors'?)
> https://snipboard.io/GQ5XRs.jpg
> vs
> https://snipboard.io/fubCc5.jpg
>  b. http://localhost:8081/cgi-bin/koha/vendors
> 'Vendors' is a link (http://localhost:8081/cgi-bin/koha/vendors#)
> 11. Interfaces
>  a. On the "show" view, "Type" is displayed when not filled (was not before)
>  b. The name of the interface is now in a `legend` tag, was h3 before. I
> don't think you can use `legend` for something else than `fieldset`.
> 12. We are missing tests. You should have a look at bug 38461 and bug 38503.

I've addressed all of the above as best I can, the only one I'm not sure on is point 12 - what specifically do you want to see tests for? The main functions are CRUD which there are cypress tests for. I've also added more unit tests for the .pm methods
Comment 270 Matt Blenkinsop 2025-02-18 14:02:20 UTC
(In reply to Jonathan Druart from comment #195)
> 13. The vendor search result is really different from what we have
> currently, and it's missing a lot of info and actions (basket list, "add to
> basket", "show all baskets", "show active vendors only", the ability to
> basket or contract for a vendor). Has this been discussed before already?

There were issues for libraries with large numbers of vendors and baskets - the vendor search could take a long time to load so the plan was to use the migration to make use of the new datatables work to allow pagination and search. The table filters allow for easy searching and filtering, with much faster load time

(In reply to Jonathan Druart from comment #196)
> The permissions store is a bit weird as you rely on a global JS variable.
> How could this be reusable from the ERM module for instance?
> 
> It would really have been great to have such concepts/ideas isolated into
> their own bug instead of lost in this huge set...

I've moved this to bug 38930

(In reply to Jonathan Druart from comment #198)
> +            `${(vendor.tax_rate * 100).toFixed(2)}` + "{enter}",
> 
> +                    label: `${(gv.option * 100).toFixed(2)}%`,
> 
> I don't think this is correct, you should use format_price (from
> format_price.inc)

Done

(In reply to Jonathan Druart from comment #202)
> Code duplication, the acquisition menu is now in
> koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc and
> koha-tmpl/intranet-tmpl/prog/js/vue/components/Islands/AcquisitionsMenu.vue
> This should be on its own bug really. Replace the left menu everywhere
> before this and don't duplicate the code.

Bug 38941

(In reply to Jonathan Druart from comment #200)
> What's the purpose of VendorDetails.vue? I don't see how it could be useful.

The vendor form component was so long and bloated, same with VendorShow. I wanted to make it more readable by splitting the html out into separate components

(In reply to Jonathan Druart from comment #204)
> Koha/Acquisition/Bookseller.pm
> +            $interface->password( $interface->plain_text_password );
> 
> Do we really want to expose the password? Maybe not something for now, but
> we should maybe have the "****" trick here with a "show" that will trigger a
> special route for plain text password (just for discussion as I am reading
> this code)

This was discussed on Mattermost and we agreed this was the best way forward for now. The documentation has been updated to make it clear that Koha is not a password manager and that passwords will be visible through the API
Comment 271 Matt Blenkinsop 2025-02-18 14:02:57 UTC
Rebased onto the tidy work, same for dependencies. QA feedback comments above
Comment 272 Marcel de Rooy 2025-03-07 10:37:13 UTC
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/js/vue/stores/vendors.js).
Comment 273 Michaela Sieber 2025-03-13 16:46:58 UTC
Soft feature freeze for 25.05 : 2025-04-29 

means 30 working days left...

What about a session in Marseille for testing and QA? 

@joubu, Matt, cait?
Comment 274 Katrin Fischer 2025-03-13 16:50:40 UTC
(In reply to Michaela Sieber from comment #273)
> Soft feature freeze for 25.05 : 2025-04-29 
> 
> means 30 working days left...
> 
> What about a session in Marseille for testing and QA? 
> 
> @joubu, Matt, cait?

I'd be happy to. Maybe Monday/Tuesday?
Comment 275 Matt Blenkinsop 2025-03-25 11:52:17 UTC
Sounds like a plan to me, I need to try and get it fully rebased before then now that i'm back
Comment 276 Jonathan Druart 2025-04-10 08:37:24 UTC
Testing on top of:
c0fc58efa3b (openfifth/bug_38010) Bug 38010: (QA follow-up) Fix toolbar on VendorShow

1. Should we provide more rewrite rule to not break existing bookmarks?
For instance http://localhost:8081/cgi-bin/koha/acqui/supplier.pl?booksellerid=1 now returns 404

2. Why do we have 
/cgi-bin/koha/vendors
instead of
/cgi-bin/koha/acquisition/vendors
?

3. breadcrumb:
http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
Click "Vendors" in the breadcrumb:
http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor#
Expected: see all vendors
Not a regression, and not sure we can fix here.

4. Filtering:
http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
"Showing 1 to 1 of 1 entries (filtered from 3 total entries)"
But you have no way to remove the filtering and see all vendors
It is the current behaviour in main, but at least we had "Search for vendor my vendor" in the breadcrumb

5. http://localhost:8081/cgi-bin/koha/acqui/vendors.pl
interestingly shows: https://snipboard.io/5Pmeq2.jpg

6. http://localhost:8081/cgi-bin/koha/vendors/2
Warning in the console:
[Vue Router warn]: Discarded invalid param(s) "vendor_id" when navigating.

7. http://localhost:8081/cgi-bin/koha/acqui/booksellers.pl?booksellerid=2
Errors in the console:
Uncaught (in promise) DOMException: CustomElementRegistry.define: 'vendor-menu' has already been defined as a custom element
Uncaught (in promise) DOMException: CustomElementRegistry.define: 'acquisitions-menu' has already been defined as a custom element

8. show vendor view:
 a. website is a button: https://snipboard.io/16ZLPA.jpg (same for the interfaces)
 b. contact options does not display correctly: https://snipboard.io/UI8Ko6.jpg
 c. "New" menu: the entries does not have the dropdown-item class => incorrect styling on mouse over: https://snipboard.io/aiB8zw.jpg
 c. Entries in the "New" menu open in new tab (same for "Receive shipments")

9. edit vendor:
 a. hit enter when you add an alias: the form is submitted. This is a regression, we prevented that in the current code with: acqui/supplier.tt:596             nodes.append("<input id='new_alias' type='text' class='noEnterSubmit' />");
 b. enter "foo" in discount: cannot save, no error on the UI, 400 in the response: {"errors":[{"message":"Expected number\/null - got string.","path":"\/body\/discount"}],"status":400}
 c. enter 111 in the discount, cannot save, no error on the UI, 500 in the response. Koha log has: Out of range value for column 'discount'
 d. delete an interface not working: "Uncaught TypeError: _ctx.deleteContact is not a function"

10. cannot edit from the list view: Uncaught Error: Missing required param "id"
Comment 277 Jonathan Druart 2025-04-10 10:25:46 UTC
11. We lost test on email and URI fields:
Account email: Please enter a valid email address.
Comment 278 Jonathan Druart 2025-04-10 12:30:23 UTC
QA review

1. We really should move C4::Contracts to Koha::Acquisition::Contract[s] before. It does not make sense to use C4::Contracts from Koha::Acquisition. The module is pretty trivial to replace.

2. I had the feeling there was something wrong in Koha::Acq::Bookseller::to_api, I think it causes this bug:
Edit a vendor, add 2 interfaces, `select * from aqbookseller_interfaces`
Edit again, don't change anything, save, select again

MariaDB [koha_kohadev]> select interface_id, vendor_id, password from aqbookseller_interfaces\G
*************************** 1. row ***************************
interface_id: 8
   vendor_id: 2
    password: 53616c7465645f5fb2f929cb1abe15d15f9a6c54e9b38977a8055ece428f2c75
*************************** 2. row ***************************
interface_id: 9
   vendor_id: 2
    password: 53616c7465645f5f2919e4566b6dacdaf29690ff2193f006bd55e843cb6ee0ae
2 rows in set (0.001 sec)

MariaDB [koha_kohadev]> select interface_id, vendor_id, password from aqbookseller_interfaces\G
*************************** 1. row ***************************
interface_id: 10
   vendor_id: 2
    password: 53616c7465645f5f0b3f2ebce9124cd14317d4fe111dd0aeedaf9532460e9e43
*************************** 2. row ***************************
interface_id: 11
   vendor_id: 2
    password: 53616c7465645f5ffd9ed9b565602e77db7510110fde8cfb43af971d46aefa3f

No changes expected, we should not add another id.

3. We lost the "Show" password link
+                            <p><span>Password: <span class="password"><a href="#" class="show_password" data-plain-text-password="[% i.plain_text_password | html %]">Show</a></span></span></p>

We discussed it previously, but I actually didn't notice that this is something we have currently. Need to be re-discussed maybe. We also had bug 34778 since then.

4. (In reply to Jonathan Druart from comment #205)
> Koha/Schema/Result/Aqbookseller.pm
> +__PACKAGE__->has_many(
> +    "aliases",
> +    "Koha::Schema::Result::AqbooksellerAlias",
> 
> Why?
> 
> We already have Koha::Acquisition::Bookseller->aliases, when is this used?

Still valid.

5. Koha/REST/V1/Acquisitions/Vendors.pm
 92     my $subscriptions = delete $vendor->{subscriptions};
 93     my $baskets       = delete $vendor->{baskets};
 94     my $contracts     = delete $vendor->{contracts};
Why?

6. Manual link
This needs to be adjusted
Koha/Manual.pm:    'acqui/supplier'                           => '/acquisitions.html#vendors',

7. DELETE /vendors/:id should use apiclient (acq.js)

8. acqui/booksellers | supplier
We pass the idea
118     supplier       => $booksellerid,
But consider it a vendor object in the template:
12         [% tx("Search for vendor {vendor}", { vendor = supplier }) | html %]
35             <span>Search for vendor [% supplier | $HtmlTags tag='em' %]</span>

8. vendors.tt
  a. +<div id="__vendors">
is this "__" to avoid conflict?
  b. I don't think we need that:
+        [% SET columns = ['cardnumber','name','category','branch','action'] %]
+        [% PROCESS patron_search_modal columns => columns, modal_title => t("Select user") %]
+        [% PROCESS patron_search_js columns => columns, filter => erm_users, actions => ["select"], preview_on_name_click => 1 %]

9. store/vendors has config.edifact = false
But we actually have more in the config: userPermissions, marcOrderAutomation, currencies, gstValues
I think this should be moved to the config route anyway (bug 38930)

10. Contracts table on the vendor show view is not correctly styled: https://snipboard.io/QHdc8N.jpg

11. i18n problems in VendorList:
+                                  escape_str(
+                                      `${row.baskets.length} basket(s)`
+                                  ) +

+                                  escape_str(
+                                      `${row.subscriptions_count} subscription(s)`
+                                  ) +

and missing escape_str
+                                  row.name +

12: 2 QA failures:
 FAIL   acqui/booksellers.pl                                                                              
   FAIL   critic                                                                                          
                # Variables::ProhibitUnusedVariables: Got 1 violation(s).                                 
                                                                                                          
 FAIL   acqui/vendors.pl                                                                                  
   FAIL   file permissions                                                                                
                File must have the exec flag
Comment 279 Jonathan Druart 2025-04-10 12:37:58 UTC
(In reply to Jonathan Druart from comment #278)
> QA review

> 2. I had the feeling there was something wrong in
> Koha::Acq::Bookseller::to_api, I think it causes this bug:
> Edit a vendor, add 2 interfaces, `select * from aqbookseller_interfaces`
> Edit again, don't change anything, save, select again
> 
> MariaDB [koha_kohadev]> select interface_id, vendor_id, password from
> aqbookseller_interfaces\G
> *************************** 1. row ***************************
> interface_id: 8
>    vendor_id: 2
>     password:
> 53616c7465645f5fb2f929cb1abe15d15f9a6c54e9b38977a8055ece428f2c75
> *************************** 2. row ***************************
> interface_id: 9
>    vendor_id: 2
>     password:
> 53616c7465645f5f2919e4566b6dacdaf29690ff2193f006bd55e843cb6ee0ae
> 2 rows in set (0.001 sec)
> 
> MariaDB [koha_kohadev]> select interface_id, vendor_id, password from
> aqbookseller_interfaces\G
> *************************** 1. row ***************************
> interface_id: 10
>    vendor_id: 2
>     password:
> 53616c7465645f5f0b3f2ebce9124cd14317d4fe111dd0aeedaf9532460e9e43
> *************************** 2. row ***************************
> interface_id: 11
>    vendor_id: 2
>     password:
> 53616c7465645f5ffd9ed9b565602e77db7510110fde8cfb43af971d46aefa3f
> 
> No changes expected, we should not add another id.

Hum, retested this on main and it's the current behaviour, not blocker!
Comment 280 Matt Blenkinsop 2025-04-23 09:40:18 UTC
(In reply to Jonathan Druart from comment #278)
> QA review
> 
> 1. We really should move C4::Contracts to Koha::Acquisition::Contract[s]
> before. It does not make sense to use C4::Contracts from Koha::Acquisition.
> The module is pretty trivial to replace.

I can do this but I think it should be a separate follow-up bug so that the change isn't lost in this code

> 3. We lost the "Show" password link
> +                            <p><span>Password: <span class="password"><a
> href="#" class="show_password" data-plain-text-password="[%
> i.plain_text_password | html %]">Show</a></span></span></p>
> 
> We discussed it previously, but I actually didn't notice that this is
> something we have currently. Need to be re-discussed maybe. We also had bug
> 34778 since then.

I think we should have a Vue component for handling this - I can add one but again that should probably be a separate follow-up

 
> 4. (In reply to Jonathan Druart from comment #205)
> > Koha/Schema/Result/Aqbookseller.pm
> > +__PACKAGE__->has_many(
> > +    "aliases",
> > +    "Koha::Schema::Result::AqbooksellerAlias",
> > 
> > Why?
> > 
> > We already have Koha::Acquisition::Bookseller->aliases, when is this used?
> 
> Still valid.

This was so that I could embed directly via the API - I couldn't seem to get it to work with just the method itself. Maybe I'm missing something?
 
> 5. Koha/REST/V1/Acquisitions/Vendors.pm
>  92     my $subscriptions = delete $vendor->{subscriptions};
>  93     my $baskets       = delete $vendor->{baskets};
>  94     my $contracts     = delete $vendor->{contracts};
> Why?

They need to be deleted before we store otherwise the store operation fails - I've changed them to just be delete clauses with no variable declaration

> 6. Manual link
> This needs to be adjusted
> Koha/Manual.pm:    'acqui/supplier'                           =>
> '/acquisitions.html#vendors',

Updated - we might need a new bug for an issue here as any url parameters in the Vue modules break the links - same happens in ERM, you just get the manual homepage

> 7. DELETE /vendors/:id should use apiclient (acq.js)

This feels like a follow-up as well, not directly related to the vendors code

> 8. acqui/booksellers | supplier
> We pass the idea
> 118     supplier       => $booksellerid,
> But consider it a vendor object in the template:
> 12         [% tx("Search for vendor {vendor}", { vendor = supplier }) | html
> %]
> 35             <span>Search for vendor [% supplier | $HtmlTags tag='em'
> %]</span>

I've changed the variable to booksellerid - we don't use the supplier object now that it is just for baskets

> 8. vendors.tt
>   a. +<div id="__vendors">
> is this "__" to avoid conflict?
>   b. I don't think we need that:
> +        [% SET columns = ['cardnumber','name','category','branch','action']
> %]
> +        [% PROCESS patron_search_modal columns => columns, modal_title =>
> t("Select user") %]
> +        [% PROCESS patron_search_js columns => columns, filter =>
> erm_users, actions => ["select"], preview_on_name_click => 1 %]

a. Changed to just be vendors
b. Deleted these as they're redundant

> 9. store/vendors has config.edifact = false
> But we actually have more in the config: userPermissions,
> marcOrderAutomation, currencies, gstValues
> I think this should be moved to the config route anyway (bug 38930)

Will upload the config route for 38930 shortly and then rebase this

> 10. Contracts table on the vendor show view is not correctly styled:
> https://snipboard.io/QHdc8N.jpg

Pager removed to match main

> 11. i18n problems in VendorList:
> +                                  escape_str(
> +                                      `${row.baskets.length} basket(s)`
> +                                  ) +
> 
> +                                  escape_str(
> +                                      `${row.subscriptions_count}
> subscription(s)`
> +                                  ) +
> 
> and missing escape_str
> +                                  row.name +

Fixed

> 12: 2 QA failures:
>  FAIL   acqui/booksellers.pl                                                
> 
>    FAIL   critic                                                            
> 
>                 # Variables::ProhibitUnusedVariables: Got 1 violation(s).   
> 
>                                                                             
> 
>  FAIL   acqui/vendors.pl                                                    
> 
>    FAIL   file permissions                                                  
> 
>                 File must have the exec flag

Fixed

I've addressed the bugs in comments 276 and 277 as well - they're all on the openfifth/bug_38010 branch as the attachment list is getting too long
Comment 281 Jonathan Druart 2025-04-23 12:42:08 UTC
(In reply to Jonathan Druart from comment #276)
> 1. Should we provide more rewrite rule to not break existing bookmarks?
> For instance
> http://localhost:8081/cgi-bin/koha/acqui/supplier.pl?booksellerid=1 now
> returns 404

Still valid.

> 3. breadcrumb:
> http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
> Click "Vendors" in the breadcrumb:
> http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor#
> Expected: see all vendors
> Not a regression, and not sure we can fix here.

Still valid, opened bug 39709.

> 4. Filtering:
> http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
> "Showing 1 to 1 of 1 entries (filtered from 3 total entries)"
> But you have no way to remove the filtering and see all vendors
> It is the current behaviour in main, but at least we had "Search for vendor
> my vendor" in the breadcrumb

Still valid, and this is confusing IMO.

> 6. http://localhost:8081/cgi-bin/koha/vendors/2
> Warning in the console:
> [Vue Router warn]: Discarded invalid param(s) "vendor_id" when navigating.

Still valid.

> 8. show vendor view:
>  a. website is a button: https://snipboard.io/16ZLPA.jpg (same for the
> interfaces)

Still a problem, the link is not clickable (missing href)

> 9. edit vendor:
>  a. hit enter when you add an alias: the form is submitted. This is a
> regression, we prevented that in the current code with:
> acqui/supplier.tt:596             nodes.append("<input id='new_alias'
> type='text' class='noEnterSubmit' />");

Fixed but now we cannot remove an alias, there is a JS error in the console: "Uncaught ReferenceError: e is not defined"

>  b. enter "foo" in discount: cannot save, no error on the UI, 400 in the
> response: {"errors":[{"message":"Expected number\/null - got
> string.","path":"\/body\/discount"}],"status":400}

Better. There is a change in the behaviour however. Before we display %.1f, now we display what is in the DB. There is only one inconsistency: 1.234 is considered valid, 1.2345 is not and the error says "Please enter a decimal number in the format: 0.0". It's not clear why 3 decimals. If it's a float it should go as it. Not considering blocker.

(In reply to Jonathan Druart from comment #277)
> 11. We lost test on email and URI fields:
> Account email: Please enter a valid email address.

Email should be fixed with only `type="email"` however the URI check is still a problem.

New ones:
12.
http://localhost:8081/cgi-bin/koha/acquisition/vendors/1
 * Missing "Receive shipment"
 * Missing space "+New"
https://snipboard.io/WpbPck.jpg

13.
Edit a vendor, open 2 contact blocks.
Tick all options for first contact, then tick all for second contact:
=> Two from the first contact are unchecked: https://snipboard.io/SLNiJI.jpg

14.
It's missing "%" for the discount display on the vendor show view: https://snipboard.io/hUw8dQ.jpg


(In reply to Matt Blenkinsop from comment #280)
> (In reply to Jonathan Druart from comment #278)
> > QA review
> > 
> > 1. We really should move C4::Contracts to Koha::Acquisition::Contract[s]
> > before. It does not make sense to use C4::Contracts from Koha::Acquisition.
> > The module is pretty trivial to replace.
> 
> I can do this but I think it should be a separate follow-up bug so that the
> change isn't lost in this code

Should have been done before this one, now it's kinda too late for 25.05.
I let you decide what to do.
Adding new uses of C4 from Koha should be avoided as much as possible.

If you do not decide to move it before this patch set, then please open a follow-up bug report.

> > 3. We lost the "Show" password link
> > +                            <p><span>Password: <span class="password"><a
> > href="#" class="show_password" data-plain-text-password="[%
> > i.plain_text_password | html %]">Show</a></span></span></p>
> > 
> > We discussed it previously, but I actually didn't notice that this is
> > something we have currently. Need to be re-discussed maybe. We also had bug
> > 34778 since then.
> 
> I think we should have a Vue component for handling this - I can add one but
> again that should probably be a separate follow-up

It's a regression caused by this change, not sure why we would need a separate bug.
If you really think we should split then do but please provide a patch ASAP.

> > 4. (In reply to Jonathan Druart from comment #205)
> > > Koha/Schema/Result/Aqbookseller.pm
> > > +__PACKAGE__->has_many(
> > > +    "aliases",
> > > +    "Koha::Schema::Result::AqbooksellerAlias",
> > > 
> > > Why?
> > > 
> > > We already have Koha::Acquisition::Bookseller->aliases, when is this used?
> > 
> > Still valid.
> 
> This was so that I could embed directly via the API - I couldn't seem to get
> it to work with just the method itself. Maybe I'm missing something?

It was invalid indeed. You need it for the order, not the embed:
_order_by=+me.name,+aliases.alias,+me.id

> > 6. Manual link
> > This needs to be adjusted
> > Koha/Manual.pm:    'acqui/supplier'                           =>
> > '/acquisitions.html#vendors',
> 
> Updated - we might need a new bug for an issue here as any url parameters in
> the Vue modules break the links - same happens in ERM, you just get the
> manual homepage

Can you open a bug report please?

> > 7. DELETE /vendors/:id should use apiclient (acq.js)
> 
> This feels like a follow-up as well, not directly related to the vendors code

It's done now, thanks!
Comment 282 Matt Blenkinsop 2025-04-23 15:41:07 UTC
(In reply to Jonathan Druart from comment #281)
> (In reply to Jonathan Druart from comment #276)
> > 1. Should we provide more rewrite rule to not break existing bookmarks?
> > For instance
> > http://localhost:8081/cgi-bin/koha/acqui/supplier.pl?booksellerid=1 now
> > returns 404
> 
> Still valid.

Fixed - redirect added to the new show page

> 
> > 4. Filtering:
> > http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
> > "Showing 1 to 1 of 1 entries (filtered from 3 total entries)"
> > But you have no way to remove the filtering and see all vendors
> > It is the current behaviour in main, but at least we had "Search for vendor
> > my vendor" in the breadcrumb
> 
> Still valid, and this is confusing IMO.

See 'Bug 38010: (QA follow-up) Stop breadcrumbs from keeping filters' for a concept - allows us to reuse it for future breadcrumbs as well

 
> > 6. http://localhost:8081/cgi-bin/koha/vendors/2
> > Warning in the console:
> > [Vue Router warn]: Discarded invalid param(s) "vendor_id" when navigating.
> 
> Still valid.

This is still happening in all Vue apps - I'm not entirely sure why, needs some deeper investigation
 
> > 8. show vendor view:
> >  a. website is a button: https://snipboard.io/16ZLPA.jpg (same for the
> > interfaces)
> 
> Still a problem, the link is not clickable (missing href)

Typo - fixed 

> > 9. edit vendor:
> >  a. hit enter when you add an alias: the form is submitted. This is a
> > regression, we prevented that in the current code with:
> > acqui/supplier.tt:596             nodes.append("<input id='new_alias'
> > type='text' class='noEnterSubmit' />");
> 
> Fixed but now we cannot remove an alias, there is a JS error in the console:
> "Uncaught ReferenceError: e is not defined"

Fixed

> (In reply to Jonathan Druart from comment #277)
> > 11. We lost test on email and URI fields:
> > Account email: Please enter a valid email address.
> 
> Email should be fixed with only `type="email"` however the URI check is
> still a problem.

I've added the type - do you mean the URI field on the form for interfaces or the email more generally?

> New ones:
> 12.
> http://localhost:8081/cgi-bin/koha/acquisition/vendors/1
>  * Missing "Receive shipment"
>  * Missing space "+New"
> https://snipboard.io/WpbPck.jpg

Can't reproduce the first, Receive shipment shows for me - did you delete the baskets for the vendor?
Space added to the "New" button

> 13.
> Edit a vendor, open 2 contact blocks.
> Tick all options for first contact, then tick all for second contact:
> => Two from the first contact are unchecked: https://snipboard.io/SLNiJI.jpg

This is the expected behaviour as we should only have one primary contact of each type - supplier.tt line 700 in main

> 14.
> It's missing "%" for the discount display on the vendor show view:
> https://snipboard.io/hUw8dQ.jpg

Fixed

> (In reply to Matt Blenkinsop from comment #280)
> > (In reply to Jonathan Druart from comment #278)
> > > QA review
> > > 
> > > 1. We really should move C4::Contracts to Koha::Acquisition::Contract[s]
> > > before. It does not make sense to use C4::Contracts from Koha::Acquisition.
> > > The module is pretty trivial to replace.
> > 
> > I can do this but I think it should be a separate follow-up bug so that the
> > change isn't lost in this code
> 
> Should have been done before this one, now it's kinda too late for 25.05.
> I let you decide what to do.
> Adding new uses of C4 from Koha should be avoided as much as possible.
> 
> If you do not decide to move it before this patch set, then please open a
> follow-up bug report.

Follow-up opened (39711) - I've already started the code for it

> > > 3. We lost the "Show" password link
> > > +                            <p><span>Password: <span class="password"><a
> > > href="#" class="show_password" data-plain-text-password="[%
> > > i.plain_text_password | html %]">Show</a></span></span></p>
> > > 
> > > We discussed it previously, but I actually didn't notice that this is
> > > something we have currently. Need to be re-discussed maybe. We also had bug
> > > 34778 since then.
> > 
> > I think we should have a Vue component for handling this - I can add one but
> > again that should probably be a separate follow-up
> 
> It's a regression caused by this change, not sure why we would need a
> separate bug.
> If you really think we should split then do but please provide a patch ASAP.
> 

I've added a patch to the branch which gives the 'Show password' functionality

> > > 6. Manual link
> > > This needs to be adjusted
> > > Koha/Manual.pm:    'acqui/supplier'                           =>
> > > '/acquisitions.html#vendors',
> > 
> > Updated - we might need a new bug for an issue here as any url parameters in
> > the Vue modules break the links - same happens in ERM, you just get the
> > manual homepage
> 
> Can you open a bug report please?

Done - bug 37912
Comment 283 Matt Blenkinsop 2025-04-28 09:17:36 UTC
Created attachment 181579 [details] [review]
Bug 38010: (QA follow-up) Add a 'Show password' option
Comment 284 Matt Blenkinsop 2025-04-28 09:19:04 UTC
Attached the url now rather than having so many patches on the bug
Comment 285 Jonathan Druart 2025-04-28 10:38:00 UTC Comment hidden (obsolete)
Comment 286 Jonathan Druart 2025-04-28 14:49:54 UTC
(In reply to Matt Blenkinsop from comment #282)
> (In reply to Jonathan Druart from comment #281)
> > > 4. Filtering:
> > > http://localhost:8081/cgi-bin/koha/vendors?supplier=my+vendor
> > > "Showing 1 to 1 of 1 entries (filtered from 3 total entries)"
> > > But you have no way to remove the filtering and see all vendors
> > > It is the current behaviour in main, but at least we had "Search for vendor
> > > my vendor" in the breadcrumb
> > 
> > Still valid, and this is confusing IMO.
> 
> See 'Bug 38010: (QA follow-up) Stop breadcrumbs from keeping filters' for a
> concept - allows us to reuse it for future breadcrumbs as well

Great. Still not perfect however, IMO we need a way to reset the filters near the table. But it's a good start. We will see what end-users say.

>  
> > > 6. http://localhost:8081/cgi-bin/koha/vendors/2
> > > Warning in the console:
> > > [Vue Router warn]: Discarded invalid param(s) "vendor_id" when navigating.
> > 
> > Still valid.
> 
> This is still happening in all Vue apps - I'm not entirely sure why, needs
> some deeper investigation

Opened bug 39764.

> > (In reply to Jonathan Druart from comment #277)
> > > 11. We lost test on email and URI fields:
> > > Account email: Please enter a valid email address.
> > 
> > Email should be fixed with only `type="email"` however the URI check is
> > still a problem.
> 
> I've added the type - do you mean the URI field on the form for interfaces
> or the email more generally?

The 2 email fields (interface and contacts) are now solved with type="email".
However we lost the URI check on "Company's details: Website".
Not blocker, not considering it really important for now.

> > New ones:
> > 12.
> > http://localhost:8081/cgi-bin/koha/acquisition/vendors/1
> >  * Missing "Receive shipment"
> >  * Missing space "+New"
> > https://snipboard.io/WpbPck.jpg
> 
> Can't reproduce the first, Receive shipment shows for me - did you delete
> the baskets for the vendor?

Oops, yes I guess so! Forget that.

> > 13.
> > Edit a vendor, open 2 contact blocks.
> > Tick all options for first contact, then tick all for second contact:
> > => Two from the first contact are unchecked: https://snipboard.io/SLNiJI.jpg
> 
> This is the expected behaviour as we should only have one primary contact of
> each type - supplier.tt line 700 in main

Indeed, it's confusing :D Sorry!
Forget that as well.

All good for me. I will do another round of code review and test tomorrow.
Comment 287 Jonathan Druart 2025-04-28 14:52:47 UTC
(In reply to Matt Blenkinsop from comment #282)
> (In reply to Jonathan Druart from comment #281)
> > > > 6. Manual link
> > > > This needs to be adjusted
> > > > Koha/Manual.pm:    'acqui/supplier'                           =>
> > > > '/acquisitions.html#vendors',
> > > 
> > > Updated - we might need a new bug for an issue here as any url parameters in
> > > the Vue modules break the links - same happens in ERM, you just get the
> > > manual homepage
> > 
> > Can you open a bug report please?
> 
> Done - bug 37912

Actually bug 39712.
Comment 288 Michaela Sieber 2025-04-29 08:16:22 UTC
tested and from an end-user perspective no blockers for the upcoming release.
Minor enhancements could be solved as follow-up bugs
Comment 289 Jonathan Druart 2025-04-29 08:28:35 UTC
(In reply to Michaela Sieber from comment #288)
> tested and from an end-user perspective no blockers for the upcoming release.
> Minor enhancements could be solved as follow-up bugs

Did you note the minor enhancements you would like to see?
Comment 290 Jonathan Druart 2025-04-30 19:30:14 UTC
Changes in behaviour: new a vendor cannot be deleted if it has invoices linked to it.
Comment 291 Jonathan Druart 2025-04-30 20:58:39 UTC
Created attachment 181727 [details] [review]
Bug 38010: Add vue app and list component

This patch adds the base vue app as well as the component to list vendors

Amended-by: Jonathan Druart
chmod 755 acqui/vendors.pl

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 292 Jonathan Druart 2025-04-30 20:58:44 UTC
Created attachment 181728 [details] [review]
Bug 38010: Add the component to display a vendor

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 293 Jonathan Druart 2025-04-30 20:58:50 UTC
Created attachment 181729 [details] [review]
Bug 38010: Add permissions

This patch adds a permissions store and utility methods for implementing permissions checks in a Vue app

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 294 Jonathan Druart 2025-04-30 20:58:56 UTC
Created attachment 181730 [details] [review]
Bug 38010: Add authorised values

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 295 Jonathan Druart 2025-04-30 20:59:00 UTC
Created attachment 181731 [details] [review]
Bug 38010: Rename routes to acquisitions

This patch renames the app from vendors to acquisitions to allow further acquisitions pages to be migrated

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 296 Jonathan Druart 2025-04-30 20:59:05 UTC
Created attachment 181732 [details] [review]
Bug 38010: Add the form to add a new vendor

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 297 Jonathan Druart 2025-04-30 20:59:10 UTC
Created attachment 181733 [details] [review]
Bug 38010: Update navigation links

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 298 Jonathan Druart 2025-04-30 20:59:15 UTC
Created attachment 181734 [details] [review]
Bug 38010: Allow external breadcrumbs

This patch allows breadcrumbs to navigate to pages outside the Vue application

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 299 Jonathan Druart 2025-04-30 20:59:19 UTC
Created attachment 181735 [details] [review]
Bug 38010: Add a contracts table to the vendor show component

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 300 Jonathan Druart 2025-04-30 20:59:24 UTC
Created attachment 181736 [details] [review]
Bug 38010: Remove drawCallback from contracts table

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 301 Jonathan Druart 2025-04-30 20:59:29 UTC
Created attachment 181737 [details] [review]
Bug 38010: Add permissions to action buttons

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 302 Jonathan Druart 2025-04-30 20:59:34 UTC
Created attachment 181738 [details] [review]
Bug 38010: Make the delete button conditional on whether a vendor has baskets or subscriptions

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 303 Jonathan Druart 2025-04-30 20:59:39 UTC
Created attachment 181739 [details] [review]
Bug 38010: Add cypress tests

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 304 Jonathan Druart 2025-04-30 20:59:45 UTC
Created attachment 181740 [details] [review]
Bug 38010: Adjust vendor update logic

This patch handles the embedded data that might also need updating when editing a vendor

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 305 Jonathan Druart 2025-04-30 20:59:49 UTC
Created attachment 181741 [details] [review]
Bug 38010: Fix vendors search bar

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 306 Jonathan Druart 2025-04-30 20:59:54 UTC
Created attachment 181742 [details] [review]
Bug 38010: Update links to vendor module

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 307 Jonathan Druart 2025-04-30 20:59:59 UTC
Created attachment 181743 [details] [review]
Bug 38010: Rebase for Button component change

Bug 38010: Fix rspack error in VendorShow

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 308 Jonathan Druart 2025-04-30 21:00:04 UTC
Created attachment 181744 [details] [review]
Bug 38010: Add basket page

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 309 Jonathan Druart 2025-04-30 21:00:09 UTC
Created attachment 181745 [details] [review]
Bug 38010: Allow embedding for baskets table

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 310 Jonathan Druart 2025-04-30 21:00:14 UTC
Created attachment 181746 [details] [review]
Bug 38010: Point basket action buttons to correct pages

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 311 Jonathan Druart 2025-04-30 21:00:19 UTC
Created attachment 181747 [details] [review]
Bug 38010: Fix null basket names

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 312 Jonathan Druart 2025-04-30 21:00:25 UTC
Created attachment 181748 [details] [review]
Bug 38010: Respect AcqViewBaskets syspref

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 313 Jonathan Druart 2025-04-30 21:00:29 UTC
Created attachment 181749 [details] [review]
Bug 38010: Add delete contact button

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 314 Jonathan Druart 2025-04-30 21:00:35 UTC
Created attachment 181750 [details] [review]
Bug 38010: Add menu components for vendor and acquisitions menus

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 315 Jonathan Druart 2025-04-30 21:00:40 UTC
Created attachment 181751 [details] [review]
Bug 38010: Implement side menus in vendors app

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 316 Jonathan Druart 2025-04-30 21:00:45 UTC
Created attachment 181752 [details] [review]
Bug 38010: Allow searching by alias in the vendors table

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 317 Jonathan Druart 2025-04-30 21:00:50 UTC
Created attachment 181753 [details] [review]
Bug 38010: Form fixes

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 318 Jonathan Druart 2025-04-30 21:00:56 UTC
Created attachment 181754 [details] [review]
Bug 38010: Rebase for bug 38390

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 319 Jonathan Druart 2025-04-30 21:01:01 UTC
Created attachment 181755 [details] [review]
Bug 38010: Add receive shipments action to table

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 320 Jonathan Druart 2025-04-30 21:01:06 UTC
Created attachment 181756 [details] [review]
Bug 38010: Restore new basket/contract buttons

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 321 Jonathan Druart 2025-04-30 21:01:10 UTC
Created attachment 181757 [details] [review]
Bug 38010: Add dropdown button component

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 322 Jonathan Druart 2025-04-30 21:01:16 UTC
Created attachment 181758 [details] [review]
Bug 38010: Reset basket view

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 323 Jonathan Druart 2025-04-30 21:01:21 UTC
Created attachment 181759 [details] [review]
Bug 38010: Remove unnecessary navigation check

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 324 Jonathan Druart 2025-04-30 21:01:26 UTC
Created attachment 181760 [details] [review]
Bug 38010: Fix tax rate display

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 325 Jonathan Druart 2025-04-30 21:01:31 UTC
Created attachment 181761 [details] [review]
Bug 38010: Add callbacks to Toolbarbutton

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 326 Jonathan Druart 2025-04-30 21:01:36 UTC
Created attachment 181762 [details] [review]
Bug 38010: Fix acqui-home.pl breadcrumbs

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 327 Jonathan Druart 2025-04-30 21:01:41 UTC
Created attachment 181763 [details] [review]
Bug 38010: Fix programmatic navigation id attribute

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 328 Jonathan Druart 2025-04-30 21:01:46 UTC
Created attachment 181764 [details] [review]
Bug 38010: Remove unneccessary test

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 329 Jonathan Druart 2025-04-30 21:01:52 UTC
Created attachment 181765 [details] [review]
Bug 38010: Delete old code

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 330 Jonathan Druart 2025-04-30 21:01:56 UTC
Created attachment 181766 [details] [review]
Bug 38010: Fix navigation links

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 331 Jonathan Druart 2025-04-30 21:02:01 UTC
Created attachment 181767 [details] [review]
Bug 38010: Prevent vendor contacts from duplicating

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 332 Jonathan Druart 2025-04-30 21:02:06 UTC
Created attachment 181768 [details] [review]
Bug 38010: Check for baskets for receive shipments button

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 333 Jonathan Druart 2025-04-30 21:02:11 UTC
Created attachment 181769 [details] [review]
Bug 38010: Check for invoices before allowing vendor deletion

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 334 Jonathan Druart 2025-04-30 21:02:16 UTC
Created attachment 181770 [details] [review]
Bug 38010: Only allow one primary contact

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 335 Jonathan Druart 2025-04-30 21:02:21 UTC
Created attachment 181771 [details] [review]
Bug 38010: Remove unnecessary buttons

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 336 Jonathan Druart 2025-04-30 21:02:25 UTC
Created attachment 181772 [details] [review]
Bug 38010: Apply format_date to vendor contracts

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 337 Jonathan Druart 2025-04-30 21:02:30 UTC
Created attachment 181773 [details] [review]
Bug 38010: Change how interface details are displayed

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 338 Jonathan Druart 2025-04-30 21:02:35 UTC
Created attachment 181774 [details] [review]
Bug 38010: (QA follow-up) Template fixes

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 339 Jonathan Druart 2025-04-30 21:02:41 UTC
Created attachment 181775 [details] [review]
Bug 38010: (QA follow-up) Fix new vendor form

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 340 Jonathan Druart 2025-04-30 21:02:47 UTC
Created attachment 181776 [details] [review]
Bug 38010: (QA follow-up) Restore navigation links

This patch restores any links that previously pointed to booksellers.pl as this script is still in use. All references to supplier.pl remain pointed at the new Vue app

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 341 Jonathan Druart 2025-04-30 21:02:53 UTC
Created attachment 181777 [details] [review]
Bug 38010: (QA follow-up) Fix delete button

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 342 Jonathan Druart 2025-04-30 21:02:59 UTC
Created attachment 181778 [details] [review]
Bug 38010: (QA follow-up) Fix vendor show component

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 343 Jonathan Druart 2025-04-30 21:03:05 UTC
Created attachment 181779 [details] [review]
Bug 38010: (QA follow-up) Use format_price

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 344 Jonathan Druart 2025-04-30 21:03:10 UTC
Created attachment 181780 [details] [review]
Bug 38010: (QA follow-up) Rebase acquisitions amd vendor menus

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 345 Jonathan Druart 2025-04-30 21:03:16 UTC
Created attachment 181781 [details] [review]
Bug 38010: (QA follow-up) Remove unnecessary code from booksellers.pl

Bug 38010: (QA follow-up) Fix QA tests

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 346 Jonathan Druart 2025-04-30 21:03:22 UTC
Created attachment 181782 [details] [review]
Bug 38010: (QA follow-up) Rebase for bug 37930

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 347 Jonathan Druart 2025-04-30 21:03:26 UTC
Created attachment 181783 [details] [review]
Bug 38010: (QA follow-up) Rebase for bug 38854

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 348 Jonathan Druart 2025-04-30 21:03:32 UTC
Created attachment 181784 [details] [review]
Bug 38010: (QA follow-up) Convert vendor menu to Vue island

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 349 Jonathan Druart 2025-04-30 21:03:37 UTC
Created attachment 181785 [details] [review]
Bug 38010: (QA follow-up) Add unit tests

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 350 Jonathan Druart 2025-04-30 21:03:42 UTC
Created attachment 181786 [details] [review]
Bug 38010: (QA follow-up) Fix toolbar on VendorShow

The delete button was missing from VendorShow and bug 37301 has been removed as a dependency so some small fixes were required for that

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 351 Jonathan Druart 2025-04-30 21:03:48 UTC
Created attachment 181787 [details] [review]
Bug 38010: (QA follow-up) Update routes to include 'acquisition'

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 352 Jonathan Druart 2025-04-30 21:03:54 UTC
Created attachment 181788 [details] [review]
Bug 38010: (QA follow-up) Redirect Home component to List

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 353 Jonathan Druart 2025-04-30 21:03:59 UTC
Created attachment 181789 [details] [review]
Bug 38010: (QA follow-up) Move island hydration to container wrapper

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 354 Jonathan Druart 2025-04-30 21:04:05 UTC
Created attachment 181790 [details] [review]
Bug 38010: (QA follow-up) Fix issues in Show component

This patch addresses the following:
- website links were buttons so we now can pass a css class to the toolbar buttons again
- contact options have been formatted to have the correct margin
- the dropdown-item class has been added to the dropdown button list
- entries in the dropdown were opening in a new tab, they are now a callback to avoid this

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 355 Jonathan Druart 2025-04-30 21:04:10 UTC
Created attachment 181791 [details] [review]
Bug 38010: (QA follow-up) Fix issues on vendor edit page

This commit addresses the following:
- Prevents form submission when hitting enter in the alias field
- validation for the discount field
- restores delete functionality to interfaces
- fixes the broken edit button on the list view

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 356 Jonathan Druart 2025-04-30 21:04:14 UTC
Created attachment 181792 [details] [review]
Bug 38010: (QA follow-up) Adjust mapping for the manual

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 357 Jonathan Druart 2025-04-30 21:04:20 UTC
Created attachment 181793 [details] [review]
Bug 38010: (QA follow-up) UI fixes

This patch addresses
- Translation issues in VendorList
- Removing the pagers from the contracts table
- Removing redundant code from the vendors template file

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 358 Jonathan Druart 2025-04-30 21:04:26 UTC
Created attachment 181794 [details] [review]
Bug 38010: (QA follow-up) Handle form parameters in the browser

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 359 Jonathan Druart 2025-04-30 21:04:31 UTC
Created attachment 181795 [details] [review]
Bug 38010: (QA follow-up) Rebase on main

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 360 Jonathan Druart 2025-04-30 21:04:36 UTC
Created attachment 181796 [details] [review]
Bug 38010: (QA follow-up) Convert delete modal to use the API client

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 361 Jonathan Druart 2025-04-30 21:04:41 UTC
Created attachment 181797 [details] [review]
Bug 38010: (QA follow-up) Add a config endpoint for permissions

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 362 Jonathan Druart 2025-04-30 21:04:46 UTC
Created attachment 181798 [details] [review]
Bug 38010: (QA follow-up) Email type checking and UI fixes

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 363 Jonathan Druart 2025-04-30 21:04:51 UTC
Created attachment 181799 [details] [review]
Bug 38010: (QA follow-up) Stop breadcrumbs from keeping filters

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 364 Jonathan Druart 2025-04-30 21:04:56 UTC
Created attachment 181800 [details] [review]
Bug 38010: (QA follow-up) Add new apache redirect

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 365 Jonathan Druart 2025-04-30 21:05:00 UTC
Created attachment 181801 [details] [review]
Bug 38010: (QA follow-up) Add a 'Show password' option

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 366 Jonathan Druart 2025-04-30 21:05:05 UTC
Created attachment 181802 [details] [review]
Bug 38010: Improve show/hide password icon

Use font-awesome-icon tag and add "hide"
Also prevent the scrollbar to jump when the link is clicked (removing
href)

(cherry picked from commit 78876ee276cf0af6e307cd7a033bbbe2842e1dc1)

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 367 Jonathan Druart 2025-04-30 21:05:10 UTC
Created attachment 181803 [details] [review]
Bug 38010: (QA follow-up) Add a Link component

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 368 Jonathan Druart 2025-04-30 21:05:14 UTC
Created attachment 181804 [details] [review]
Bug 38010: (QA follow-up) Add missing filter to main-container.inc

Amended-by: Jonathan Druart
-[% SET islands = Asset.js("js/vue/dist/islands.esm.js").match('(src="([^"]+)")').1 | $raw %] <script src="[% islands %]" type="module">
+[% SET islands = Asset.js("js/vue/dist/islands.esm.js").match('(src="([^"]+)")').1 %] <script src="[% islands | $raw %]" type="module">

Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 369 Jonathan Druart 2025-04-30 21:05:19 UTC
Created attachment 181805 [details] [review]
Bug 38010: (QA follow-up) Remove extra sidebar_menu class

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 370 Jonathan Druart 2025-04-30 21:05:24 UTC
Created attachment 181806 [details] [review]
Bug 38010: Remove op=do_search from the 'Invoices' link

So we don't see "no result for your search"

(cherry picked from commit 1cc3dabaa25d503b6b6b7d49d81c9bf739013d10)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 371 Jonathan Druart 2025-04-30 21:05:29 UTC
Created attachment 181807 [details] [review]
Bug 38010: Code improvement

(cherry picked from commit a14f1898672950be9d9097bc7b5dd56392a241b0)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 372 Jonathan Druart 2025-04-30 21:05:34 UTC
Created attachment 181808 [details] [review]
Bug 38010: Fix links to the manual

The way we deal with routes for VueJS is messy and will need to be
reconsidered, but it's not within the scope of this patch.

(cherry picked from commit 2558583ef0fbf149664a3d9a2e3cec081ac505ff)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 373 Jonathan Druart 2025-04-30 21:05:39 UTC
Created attachment 181809 [details] [review]
Bug 38010: Fix wrong use of scalar in the REST API controller

It was wrong, we need to use scalar on the array, not the arrayref
(otherwise it will always be true)
No change in behaviour expected here, just some code cleaning

(cherry picked from commit 7abbcb7ba73da13d988202aaf06ba239e21692da)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 374 Jonathan Druart 2025-04-30 21:05:45 UTC
Created attachment 181810 [details] [review]
Bug 38010: Add tests for embed contacts,interfaces,aliases

(cherry picked from commit cae75569e6e8f2bfe5a2e6d3acc111485f81cc1e)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 375 Jonathan Druart 2025-04-30 21:05:49 UTC
Created attachment 181811 [details] [review]
Bug 38010: (QA follow-up) Move syspref values to config endpoint

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 376 Jonathan Druart 2025-04-30 21:05:55 UTC
Created attachment 181812 [details] [review]
Bug 38010: Fix LeftMenu

(cherry picked from commit a78af54eebf07a623779f4642bcdad28f35e662f)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 377 Jonathan Druart 2025-04-30 21:05:59 UTC
Created attachment 181813 [details] [review]
Bug 38010: Embed baskets+count and subscriptions+count

We don't need to whole list of baskets and subscriptions

(cherry picked from commit 391b4d3b32b57c97bfb7e66ab394020ae7eed0e5)

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 378 Jonathan Druart 2025-04-30 21:06:05 UTC
Created attachment 181814 [details] [review]
Bug 38010: Migrate to_api logic to GET endpoint

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 379 Jonathan Druart 2025-04-30 21:06:09 UTC
Created attachment 181815 [details] [review]
Bug 38010: Translation fix

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 380 Jonathan Druart 2025-04-30 21:06:15 UTC
Created attachment 181816 [details] [review]
Bug 38010: Add protection logic to the DELETE endpoint

Amended-by: Jonathan Druart
typo: subsicriptions => subscriptions

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 381 Jonathan Druart 2025-04-30 21:06:20 UTC
Created attachment 181817 [details] [review]
Bug 38010: (QA follow-up) Add default settings to vendor store

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 382 Jonathan Druart 2025-04-30 21:06:24 UTC
Created attachment 181818 [details] [review]
Bug 38010: Fix vendor deletion tests

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 383 Jonathan Druart 2025-04-30 21:06:30 UTC
Created attachment 181819 [details] [review]
Bug 38010: Add tests for DELETE vendors/{vendor_id}

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 384 Jonathan Druart 2025-04-30 21:06:35 UTC
Created attachment 181820 [details] [review]
Bug 38010: Fix cypress tests - do not intercept /config

We do not want to intercept /acquisitions/vendors/config
Ideally we should mock it separately

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 385 Jonathan Druart 2025-04-30 21:06:40 UTC
Created attachment 181821 [details] [review]
Bug 38010: Fix cypress tests - add _count to the mock object

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 386 Katrin Fischer 2025-05-02 15:18:16 UTC
95 patches... ugh.  Did some testing, found some smaller and some odder things, no deal breakers, but I believe in quick follow-ups before release!

1) Terminology (follow-up)

+    description: List of GST values

We removed GST as a term a while ago, I think this should probably read:
List of tax rates

+    description: EDIFACT syspref value
+    description: MarcOrderingAutomation syspref value

Please always spell out system preference instead of using the short form. (see Terminology list on wiki)



2) Vendor result list

a) Currency takes a lot of width, not sure why. But it's also ambiguous. Can you please clarify if it's the list price currency or invoice price currency? (follow-up, string change)

b) I think the vendor result table would greatly benefit from being column configurable (separate bug, can file later)

c) The action column should be excluded from exports and print preview. (follow-up)

d) Alias: if a vendor is found by searching for it's alias, it would be nice if there was some visual indicator. Maybe when adding column configuration we could add a column displaying the alias? (possible enhancement, can file later)



3) Managing vendors

a) Encoding problem for the interface password (separate bug)

- Create a new vendor
- Add an interface with a password: KölnHamburgBerlinMünster
- Save
- Edit: The password umlauts appear mangled
... Now I was going to mark this blocker, but it already happens in 24.11...

b) Deleting vendor only works from result list (follow-up or separate bug)

In 24.11 we have a "Delete vendor" in the toolbar you can see on the vendor's detail page. With this patch set the "Delete" link only appears in the vendor result list. It works from there, so the vendor was "deletable" (no orders or subscriptions linked).



4) Managing contracts (follow-up or separate bug)

a) Start and end date missing mark-up for sorting correctly with different date formats. This works in 24.11 (regression). The styling of the contacts table also appears a little odd, like dates should be aligned right etc.?



X) Should not always complain, bonus points for:

* Updating links to help files
* Updating POD
* Switching suppliers.pl to vendors.pl
* Translatability

:)
Comment 387 Katrin Fischer 2025-05-02 15:24:04 UTC
Pushed for 25.05!

Well done everyone, thank you!
Comment 388 Jonathan Druart 2025-05-05 07:50:20 UTC
Created attachment 181891 [details] [review]
Bug 38010: Fix xt/api.t
Comment 389 Jonathan Druart 2025-05-05 07:54:48 UTC
(In reply to Jonathan Druart from comment #388)
> Created attachment 181891 [details] [review] [review]
> Bug 38010: Fix xt/api.t

Patch pushed.
Comment 390 Jonathan Druart 2025-05-05 08:17:19 UTC
(In reply to Katrin Fischer from comment #386)
> 3) Managing vendors
> 
> a) Encoding problem for the interface password (separate bug)
> 
> - Create a new vendor
> - Add an interface with a password: KölnHamburgBerlinMünster
> - Save
> - Edit: The password umlauts appear mangled
> ... Now I was going to mark this blocker, but it already happens in 24.11...

Opened bug 39826.
Comment 391 Jonathan Druart 2025-05-05 08:34:08 UTC
Created attachment 181898 [details] [review]
Bug 38010: Embed invoices_count - fix patron deletion from show view

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 392 Jonathan Druart 2025-05-05 08:37:34 UTC
(In reply to Katrin Fischer from comment #386)
> b) Deleting vendor only works from result list (follow-up or separate bug)
> 
> In 24.11 we have a "Delete vendor" in the toolbar you can see on the
> vendor's detail page. With this patch set the "Delete" link only appears in
> the vendor result list. It works from there, so the vendor was "deletable"
> (no orders or subscriptions linked).

Done on the last patch.

(In reply to Jonathan Druart from comment #391)
> Created attachment 181898 [details] [review] [review]
> Bug 38010: Embed invoices_count - fix patron deletion from show view
> 
> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Pushed to main.
Comment 393 Jonathan Druart 2025-05-05 09:16:28 UTC
Created attachment 181901 [details] [review]
Bug 38010: Restore ordering on the contracts table
Comment 394 Matt Blenkinsop 2025-05-06 10:16:54 UTC
Created attachment 181963 [details] [review]
Bug 38010: (follow-up) Fix API spec terminology
Comment 395 Matt Blenkinsop 2025-05-06 10:16:59 UTC
Created attachment 181964 [details] [review]
Bug 38010: (follow-up) Make vendor table columns configurable
Comment 396 Katrin Fischer 2025-05-06 15:46:01 UTC
I could apply Matts 2 follow-ups, but I get an sha1 error on yours Joubu:

Apply? [(y)es, (n)o, (i)nteractive] i
Applying: Bug 38010: Restore ordering on the contracts table
error: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorContracts.vue).
error: could not build fake ancestor

It doesn't appear to be pushed either.
Comment 397 Jonathan Druart 2025-05-07 21:16:52 UTC
Created attachment 182053 [details] [review]
Bug 38010: Restore ordering on the contracts table
Comment 398 Jonathan Druart 2025-05-07 21:17:54 UTC
(In reply to Katrin Fischer from comment #396)
> I could apply Matts 2 follow-ups, but I get an sha1 error on yours Joubu:
> 
> Apply? [(y)es, (n)o, (i)nteractive] i
> Applying: Bug 38010: Restore ordering on the contracts table
> error: sha1 information is lacking or useless
> (koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorContracts.vue).
> error: could not build fake ancestor
> 
> It doesn't appear to be pushed either.

Oops yes, I had another patch on the branch, and both should have been squashed before attaching. Should be fixed now.
Comment 399 Katrin Fischer 2025-05-08 05:45:00 UTC
Pushed last follow-up. 

@Joubu: Your patch restores the sorting. But I am wondering: I don't see the mark-up for date sorting in the table. data-order with the sortable form of the date?

Applied patch, ran yarn build.
Comment 400 Matt Blenkinsop 2025-05-13 10:11:56 UTC
Created attachment 182351 [details] [review]
Bug 38010: (follow-up) Remove DT reference
Comment 401 Jonathan Druart 2025-05-13 10:22:21 UTC
(In reply to Katrin Fischer from comment #399)
> Pushed last follow-up. 
> 
> @Joubu: Your patch restores the sorting. But I am wondering: I don't see the
> mark-up for date sorting in the table. data-order with the sortable form of
> the date?
> 
> Applied patch, ran yarn build.

Looks ok to me: https://snipboard.io/1D8hHo.jpg

The trick is here:
+                            return type == "sort"
+                                ? row.contractenddate
+                                : format_date(row.contractenddate);
Comment 402 Jonathan Druart 2025-05-13 10:23:34 UTC
(In reply to Matt Blenkinsop from comment #400)
> Created attachment 182351 [details] [review] [review]
> Bug 38010: (follow-up) Remove DT reference

Patch pushed.
Comment 403 Jonathan Druart 2025-05-14 11:38:20 UTC
Created attachment 182438 [details] [review]
Bug 38010: Fix cypress tests by mocking more API responses

Jenkins runs the tests on top of the onboarding, there is no acq data
Comment 404 Jonathan Druart 2025-05-14 11:39:27 UTC
(In reply to Jonathan Druart from comment #403)
> Created attachment 182438 [details] [review] [review]
> Bug 38010: Fix cypress tests by mocking more API responses
> 
> Jenkins runs the tests on top of the onboarding, there is no acq data

This patch fixes the 2 remaining cypress failures

<testcase name="Vendor CRUD operations should edit a vendor" time="0.000" classname="should edit a vendor">
  <failure message="Timed out retrying after 10000ms: Expected to find element: `#vendors_list table tbody tr:first`, but never found it." type="AssertionError"><![CDATA[AssertionError: Timed out retrying after 10000ms: Expected to find element: `#vendors_list table tbody tr:first`, but never found it.
at Context.eval (webpack://koha/./t/cypress/integration/Acquisitions/Vendors_spec.ts:193:11)]]></failure>
</testcase>
<testcase name="Vendor CRUD operations should show a vendor" time="0.000" classname="should show a vendor">
  <failure message="Timed out retrying after 10000ms: Expected to find element: `#vendors_show h1`, but never found it." type="AssertionError"><![CDATA[AssertionError: Timed out retrying after 10000ms: Expected to find element: `#vendors_show h1`, but never found it.
at Context.eval (webpack://koha/./t/cypress/integration/Acquisitions/Vendors_spec.ts:232:11)]]></failure>
</testcase>
Comment 405 Jonathan Druart 2025-05-14 11:59:48 UTC
(In reply to Jonathan Druart from comment #403)
> Created attachment 182438 [details] [review] [review]
> Bug 38010: Fix cypress tests by mocking more API responses
> 
> Jenkins runs the tests on top of the onboarding, there is no acq data

Patch pushed.
Comment 406 Katrin Fischer 2025-05-16 15:20:32 UTC
I believe we have taken care of the lose ends here, please correct me if I missed something.