Skip to content
This repository has been archived by the owner on Nov 21, 2023. It is now read-only.

feat(admin-ui): add table, hint, form, and initial work on data table #18

Merged
merged 16 commits into from
Aug 10, 2023

Conversation

kasperkristensen
Copy link
Collaborator

What

  • Adds Table component
  • Adds Hint component
  • Adds Form block
  • Adds WIP implementation of Data Table

What are blocks
Blocks are "batteries included" components that usually will consist of multiple "dumb" components from the components folder, as well as implement some API.
This PR contains a Form block that contains components that help with setting up forms the "admin way" using react-hook-form. The components ensure that correct IDs are created and set for related components, as well as ensure that the form is accessible.
The PR also contains initial work on a DataTable block, which is a generic Table component that uses @tanstack/react-table to set up a Table that can be either controlled or uncontrolled. The implementation is still WIP so the component is not exported atm.

@vercel
Copy link

vercel bot commented Jul 25, 2023

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

Name Status Preview Comments Updated (UTC)
ui-ui ❌ Failed (Inspect) Aug 4, 2023 1:09pm

@kasperkristensen kasperkristensen marked this pull request as draft August 8, 2023 08:30
Copy link
Contributor

@StephixOne StephixOne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, though a DataTable block or at least example will be very useful whenever we can get to it, for a more batteries-included approach.

@kasperkristensen kasperkristensen marked this pull request as ready for review August 10, 2023 19:43
@kasperkristensen kasperkristensen merged commit b36fee7 into develop Aug 10, 2023
1 check passed
@kasperkristensen kasperkristensen deleted the feat/table-and-form branch August 10, 2023 19:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants