Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: users crud example #12

Merged
merged 4 commits into from
Nov 16, 2023
Merged

feature: users crud example #12

merged 4 commits into from
Nov 16, 2023

Conversation

JLin-99
Copy link

@JLin-99 JLin-99 commented Nov 16, 2023

⚡ Users CRUD ⚡

💻 What type of change is this?

  • 💎 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Styling
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build / CI

⭐ Description

I've added queries and mutations examples. Also replaced HeadlessUI with RadixUI.

image

ONLY ADD SECTION IF A NEW PACKAGE IS ADDED

Requires

This pr requires the following packages to be installed:

  • tailwindcss-animate
  • @hookform/resolvers
  • @radix-ui/react-dialog
    The packages are used for modals and forms.

Because of this, you must run npm i before starting.

✅ Checklist

  • This PR can be merged (it is not a draft, work in progress, or blocked on another PR)
  • I have tested this change locally in multiple screen sizes
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

select: (users) =>
users.map((user, idx) => {
const selectedItem =
activityItems[idx % activityItems.length] ?? activityItems[0];

Choose a reason for hiding this comment

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

what is this doing exactly?

Copy link
Author

@JLin-99 JLin-99 Nov 16, 2023

Choose a reason for hiding this comment

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

a way to show how the select works (?. It takes the users data for the tailwindUI table columns. I'm only getting name and id from the current backend and filling them with the hardcoded data.
Screenshot 2023-11-16 at 16 20 44

@JLin-99 JLin-99 merged commit 8f4d655 into 10.x Nov 16, 2023
1 of 3 checks passed
@JLin-99 JLin-99 deleted the feature/users-crud-example branch November 16, 2023 19:26
sojeda pushed a commit that referenced this pull request Nov 28, 2023
* feat: add query examples

* feat: modal example

* feat: user CRUD example

* feat: use radix
sojeda pushed a commit that referenced this pull request Jan 2, 2024
* feat: add query examples

* feat: modal example

* feat: user CRUD example

* feat: use radix
sojeda pushed a commit that referenced this pull request Jan 2, 2024
* feat: add query examples

* feat: modal example

* feat: user CRUD example

* feat: use radix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants