feat(dashboard,icons,types,js-sdk): add providers to location UI #8328
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
5 Skipped Deployments
|
|
| await queryClient.invalidateQueries({ | ||
| queryKey: stockLocationsQueryKeys.lists(), | ||
| }) |
There was a problem hiding this comment.
nit: think we can skip invalidating the lists key for this one
| <HandTruck className="text-ui-fg-subtle" /> | ||
| </IconAvatar> | ||
|
|
||
| <ListSummary |
There was a problem hiding this comment.
question: should the "+1 more" be displayed inline without spacing in between and right alignment?
There was a problem hiding this comment.
added a comment below that changes this whole thing.
| return ( | ||
| <RouteFocusModal.Form form={form}> | ||
| <div className="flex h-full flex-col overflow-hidden"> | ||
| <RouteFocusModal.Header> |
There was a problem hiding this comment.
suggestion: believe we started moving header actions to footer. Iinstead this use <RouteFocusModal.Footer> after <RouteFocusModal.Body>.
|
@fPolic just checked the designs again, it should've been a list. I've changed to that. 
|
what:
bug found:
There is a weird overlapping back button when the modal comes up. Can be seen in the video.
providers.mp4
RESOLVES CC-231