Would Medusa welcome native regression coverage for stock_location.metadata? #15434
jasonmerx
started this conversation in
Show and tell
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi Medusa team,
I have a small testing/coverage question rather than a bug report.
While comparing Medusa
2.13.6 -> 2.14.0, I noticed that top-levelstock_location.metadatais now part of the public Admin HTTP type forAdminStockLocation, apparently related to#15025, which added theStock Location metadata form in the Admin dashboard.
From a source comparison:
StockLocationalready hadmetadatain the module model.defaultAdminStockLocationFieldsalready includedmetadata.AdminStockLocation.metadata.address.metadata, but not top-levelstock_location.metadata.So I am not treating this as a backend bug or a breaking change.
My question is: would Medusa welcome a small native integration test that
locks this Admin response behavior?
The test would probably live in:
Beta Was this translation helpful? Give feedback.
All reactions