design-system icon indicating copy to clipboard operation
design-system copied to clipboard

Pagination TS conversion

Open zamoore opened this issue 1 year ago • 5 comments

:pushpin: Summary

If merged, this PR converts the Hds::Pagination component and its children to TS

:link: External links

Jira ticket: https://hashicorp.atlassian.net/browse/HDS-2707

zamoore avatar Jun 18 '24 21:06 zamoore

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
hds-showcase ✅ Ready (Inspect) Visit Preview Aug 28, 2024 4:32pm
hds-website ✅ Ready (Inspect) Visit Preview Aug 28, 2024 4:32pm

vercel[bot] avatar Jun 18 '24 21:06 vercel[bot]

@zamoore can you please move this PR to draft? the reasons I am asking are 1) we tend to avoid doing the conversion of two different/distinct components in the same PR, and 2) all the Hds::Form-related conversion will be done in the same batch (probably by @alex-ju).

PS: I am not sure if you have seen this file, it’s the list of components that need to be converted to TS, with their dependencies. If one component has a dependency, we should do the dependency first

didoo avatar Jun 24 '24 10:06 didoo

@zamoore can you please move this PR to draft? the reasons I am asking are 1) we tend to avoid doing the conversion of two different/distinct components in the same PR, and 2) all the Hds::Form-related conversion will be done in the same batch (probably by @alex-ju).

PS: I am not sure if you have seen this file, it’s the list of components that need to be converted to TS, with their dependencies. If one component has a dependency, we should do the dependency first

@didoo Sure thing, would we want to wait to merge the changes to Pagination until the dropdown work is done as part of the batched form conversion work then? Or are you comfortable with merging to dropdown work now in a separate PR?

zamoore avatar Jun 24 '24 13:06 zamoore

@zamoore can you please move this PR to draft? the reasons I am asking are 1) we tend to avoid doing the conversion of two different/distinct components in the same PR, and 2) all the Hds::Form-related conversion will be done in the same batch (probably by @alex-ju). PS: I am not sure if you have seen this file, it’s the list of components that need to be converted to TS, with their dependencies. If one component has a dependency, we should do the dependency first

@didoo Sure thing, would we want to wait to merge the changes to Pagination until the dropdown work is done as part of the batched form conversion work then? Or are you comfortable with merging to dropdown work now in a separate PR?

I would prefer to wait if you don't mind (I made the same thing here, so don't worry: https://github.com/hashicorp/design-system/pull/2094)

didoo avatar Jun 24 '24 14:06 didoo

you probably saw this @zamoore, but I ported form/select/base together with the rest of the form conversions, so this will require a rebase, then we can focus on the pagination bits

alex-ju avatar Jul 17 '24 16:07 alex-ju