Skip to content

Conversation

@BKadirkhodjaev
Copy link
Contributor

@BKadirkhodjaev BKadirkhodjaev commented Jan 27, 2026

Purpose

Approach

  • Change how items are update for instance change connection to use the new batch storage endpoint
  • Update tests

Related PRs

@BKadirkhodjaev BKadirkhodjaev marked this pull request as ready for review January 28, 2026 15:09
@BKadirkhodjaev BKadirkhodjaev requested a review from a team as a code owner January 28, 2026 15:09
@damien-git
Copy link
Contributor

@BKadirkhodjaev One important difference is that you are sending partial items. I am not finding any documentation or code saying /item-storage/batch/synchronous would merge properties from existing items with the upserted ones (I looked at mod-inventory-storage, RMB postSync() and RMB upsert()). Can you explain why sending partial items is sufficient ?

@BKadirkhodjaev
Copy link
Contributor Author

BKadirkhodjaev commented Jan 29, 2026

@BKadirkhodjaev One important difference is that you are sending partial items. I am not finding any documentation or code saying /item-storage/batch/synchronous would merge properties from existing items with the upserted ones (I looked at mod-inventory-storage, RMB postSync() and RMB upsert()). Can you explain why sending partial items is sufficient ?

Hmm, good catch. I've asked Pavlo from Spitfire to help me understand whether it can truly patch values. In case it does not, I will try to change the fetch items endpoint to retrieve directly from mod-inventory-storage to match the shape of the batch endpoint

@BKadirkhodjaev
Copy link
Contributor Author

@BKadirkhodjaev One important difference is that you are sending partial items. I am not finding any documentation or code saying /item-storage/batch/synchronous would merge properties from existing items with the upserted ones (I looked at mod-inventory-storage, RMB postSync() and RMB upsert()). Can you explain why sending partial items is sufficient ?

Pavlo suggested we use this endpoint, it supports proper patching:

@damien-git
Copy link
Contributor

Pavlo suggested we use this endpoint, it supports proper patching:

* https://s3.amazonaws.com/foliodocs/api/mod-inventory-storage/r/item-storage.html#item_storage_items_patch

Good, that makes sense and it's still a batch update.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 9, 2026

@BKadirkhodjaev BKadirkhodjaev merged commit 7526f06 into master Feb 9, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants