feat(dashboard,medusa,ui): Manual gift cards + cleanup #6380
Conversation
🦋 Changeset detectedLatest commit: a4acbd0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
|
The code cell is off. Should we truncate and add a tooltip in case it gets cut off?
|
|
The "Original amount" and "Balance" columns should be "Initial balance" and "Current balance" to be consistent with the details page. |
|
Should we attempt to render the recipient email on the details page by scanning the metadata? |
| </CommandBar> | ||
| </div> | ||
| )} | ||
| <DataTable |
There was a problem hiding this comment.
comment: Dang, looking at these refactors is always a pleasure
| queryFn: async () => medusa.admin.giftCards.retrieve(id), | ||
| }) | ||
|
|
||
| export const giftCardLoader = async ({ params }: LoaderFunctionArgs) => { |
There was a problem hiding this comment.
When the unstable_ is gone here I am pulling the trigger 🔫
Dashboard
Medusa
UI
Note for review
Since this PR contains updates to the translation keys, it touches a lot of files. For the review the parts that are relevant are: the /gift-cards domain of admin, the table overview of collections, customers, and customer groups. And the changes to the list endpoints in the core.