Skip to content

feat(dashboard,types): add metadata form to Stock Location#15025

Open
NicolasGorga wants to merge 2 commits intodevelopfrom
feat/stock-location-dashboard-metadata
Open

feat(dashboard,types): add metadata form to Stock Location#15025
NicolasGorga wants to merge 2 commits intodevelopfrom
feat/stock-location-dashboard-metadata

Conversation

@NicolasGorga
Copy link
Copy Markdown
Contributor

Summary

What — What changes are introduced in this PR?

Add metadata form to Stock Location in Admin dashboard.

Why — Why are these changes relevant or necessary?

No way to see nor update metadata from the Dashboard.

How — How have these changes been implemented?

Added metadata from in new route.

Testing — How have these changes been tested, or how can the reviewer test the feature?

Please provide answer here


Examples

Provide examples or code snippets that demonstrate how this feature works, or how it can be used in practice.
This helps with documentation and ensures maintainers can quickly understand and verify the change.

// Example usage

Checklist

Please ensure the following before requesting a review:

  • I have added a changeset for this PR
    • Every non-breaking change should be marked as a patch
    • To add a changeset, run yarn changeset and follow the prompts
  • The changes are covered by relevant tests
  • I have verified the code works as intended locally
  • I have linked the related issue(s) if applicable

Additional Context

Add any additional context, related issues, or references that might help the reviewer understand this PR.

@NicolasGorga NicolasGorga requested a review from a team as a code owner April 7, 2026 14:12
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 7, 2026

🦋 Changeset detected

Latest commit: de95bb9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 77 packages
Name Type
@medusajs/dashboard Patch
@medusajs/types Patch
@medusajs/admin-bundler Patch
@medusajs/event-bus-redis Patch
@medusajs/draft-order Patch
@medusajs/framework Patch
@medusajs/js-sdk Patch
@medusajs/modules-sdk Patch
@medusajs/orchestration Patch
@medusajs/utils Patch
@medusajs/workflows-sdk Patch
@medusajs/medusa-oas-cli Patch
integration-tests-http Patch
@medusajs/medusa Patch
@medusajs/test-utils Patch
@medusajs/analytics Patch
@medusajs/api-key Patch
@medusajs/auth Patch
@medusajs/cache-inmemory Patch
@medusajs/cache-redis Patch
@medusajs/caching Patch
@medusajs/cart Patch
@medusajs/currency Patch
@medusajs/customer Patch
@medusajs/event-bus-local Patch
@medusajs/file Patch
@medusajs/fulfillment Patch
@medusajs/index Patch
@medusajs/inventory Patch
@medusajs/link-modules Patch
@medusajs/locking Patch
@medusajs/notification Patch
@medusajs/order Patch
@medusajs/payment Patch
@medusajs/pricing Patch
@medusajs/product Patch
@medusajs/promotion Patch
@medusajs/rbac Patch
@medusajs/region Patch
@medusajs/sales-channel Patch
@medusajs/settings Patch
@medusajs/stock-location Patch
@medusajs/store Patch
@medusajs/tax Patch
@medusajs/translation Patch
@medusajs/user Patch
@medusajs/workflow-engine-inmemory Patch
@medusajs/workflow-engine-redis Patch
@medusajs/analytics-local Patch
@medusajs/analytics-posthog Patch
@medusajs/auth-emailpass Patch
@medusajs/auth-github Patch
@medusajs/auth-google Patch
@medusajs/caching-redis Patch
@medusajs/file-local Patch
@medusajs/file-s3 Patch
@medusajs/fulfillment-manual Patch
@medusajs/locking-postgres Patch
@medusajs/locking-redis Patch
@medusajs/notification-local Patch
@medusajs/notification-sendgrid Patch
@medusajs/payment-stripe Patch
@medusajs/core-flows Patch
@medusajs/oas-github-ci Patch
@medusajs/http-types-generator Patch
@medusajs/cli Patch
@medusajs/deps Patch
@medusajs/telemetry Patch
@medusajs/admin-sdk Patch
@medusajs/admin-shared Patch
@medusajs/admin-vite-plugin Patch
@medusajs/icons Patch
@medusajs/toolbox Patch
@medusajs/ui-preset Patch
create-medusa-app Patch
medusa-dev-cli Patch
@medusajs/ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

9 Skipped Deployments
Project Deployment Actions Updated (UTC)
api-reference Ignored Ignored Apr 7, 2026 2:12pm
api-reference-v2 Ignored Ignored Apr 7, 2026 2:12pm
bloom-docs Ignored Ignored Apr 7, 2026 2:12pm
cloud-docs Ignored Ignored Apr 7, 2026 2:12pm
docs-ui Ignored Ignored Apr 7, 2026 2:12pm
docs-v2 Ignored Ignored Apr 7, 2026 2:12pm
medusa-docs Ignored Ignored Apr 7, 2026 2:12pm
resources-docs Ignored Ignored Apr 7, 2026 2:12pm
user-guide Ignored Ignored Apr 7, 2026 2:12pm

Request Review

@medusa-os-bot
Copy link
Copy Markdown

medusa-os-bot bot commented Apr 9, 2026

Thank you for your contribution! 🎉

After an initial review, this PR looks good to us. Here's a summary:

✅ Changeset included
✅ Follows contribution guidelines
✅ Follows Medusa's conventions (MetadataForm + RouteDrawer pattern, matching existing metadata routes)
✅ Type addition is correct — metadata was missing from AdminStockLocation entity despite being stored in the DB

Notes:

  • All other location routes (e.g. location-edit.tsx) destructure location_id from useParams(), not id. Please confirm location-metadata.tsx uses const { location_id } = useParams() rather than const { id } = useParams() — if it copies from region-metadata or inventory-metadata as a template, this would silently pass undefined to the hook.
  • The Testing section of the PR description is still Please provide answer here — not a blocker for team members, but worth filling in for future reference.

A team member will do a final review before this is merged. We appreciate your patience!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant