[UIREC-139] align dependency versions affected by Inventory's Optimistic Locking Created: 01/Jun/21  Updated: 14/Sep/21  Resolved: 31/Aug/21

Status: Closed
Project: ui-receiving
Components: None
Affects versions: None
Fix versions: 2.0.0

Type: Story Priority: P2
Reporter: Jakub Skoczen Assignee: Julian Ladisch
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original estimate: Not Specified

Issue links:
Cloners
clones FOLIO-3179 ui-inventory-es: align dependency ver... Closed
is cloned by MODAUD-81 align dependency versions affected by... Closed
Defines
defines UXPROD-3089 Inventory. Implementing Optimistic Lo... Closed
Sprint: CP: R3 2022 roadmap, CP: sprint 115, CP: sprint 117, CP: sprint 118
Story Points: 1
Development Team: Core: Platform

 Description   

According to Okapi dependency graph:

http://localhost:9130/_/proxy/modules?require=item-storage,item-storage-batch-sync,holdings-storage,holdings-storage-batch-sync,instance-storage,instance-storace-batch,instance-storage-batch-sync&latest=1

this module depends on one of the Inventory's interfaces that requires a major version change due to introduction of Optimistic Locking (see MODINV-398 Closed and MODINVSTOR-726 Closed ).

The new dependency should support both version: previous and new one.



 Comments   
Comment by Julian Ladisch [ 20/Aug/21 ]

Mikita Siadykh Please code review and merge.

UIREC-139 Closed needs to be merged before we merge https://github.com/folio-org/mod-inventory-storage/pull/624 = https://folio-org.atlassian.net/browse/MODINVSTOR-726 = https://folio-org.atlassian.net/browse/MODINVSTOR-713

We need this for the test: https://folio-org.atlassian.net/browse/FOLIO-3241

This is a FOLIO-wide coordinated effort: https://folio-org.atlassian.net/browse/UXPROD-3089

For background see https://folio-org.atlassian.net/wiki/display/DD/Optimistic+locking+proposal

Comment by Mikita Siadykh [ 20/Aug/21 ]

Julian Ladisch approved, but not merged yet, when is it planned to merge mod-inventory-storage? I understand that before/together other PRs are merged, but it's not accurate value

Comment by Julian Ladisch [ 20/Aug/21 ]

Mikita Siadykh: Thank you for review, please merge to master.

This pull request ( https://github.com/folio-org/ui-receiving/pull/281 ) is compatible with old (without optimistic locking) and new (with optimistic locking) interfaces of mod-inventory-storage, it lists both interface versions. Therefore it can be merged to master at any time, no need to wait until mod-inventory-storage merges the change.

The core-platform wants to run an optimistic locking test on its scratch environment next week: https://folio-org.atlassian.net/browse/FOLIO-3241 It is much easier if as many module as possible have merged their optimistic locking pull requests to master.

Therefore the core-platform team asks the Thunderjet team to merge this pull request to master (core-platform team does not have write permission to ui-receiving repository).

Comment by Mikita Siadykh [ 20/Aug/21 ]

Julian Ladisch done

Comment by Julian Ladisch [ 20/Aug/21 ]

Thanks!

Generated at Thu Feb 08 23:26:12 UTC 2024 using Jira 1001.0.0-SNAPSHOT#100246-sha1:7a5c50119eb0633d306e14180817ddef5e80c75d.