-
-
Notifications
You must be signed in to change notification settings - Fork 565
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
feat: initial support for lists #1474
Conversation
Run & review this pull request in StackBlitz Codeflow. |
✅ Deploy Preview for elk-zone ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for elk-docs canceled.
|
doesn't work, and I don't know if they are even good to have
doesn't work, and I don't know if they are even good to have
@patak-dev I'd call this "ready for review". I've done testing and everything seems to be working, but future pull requests will include the ability to view the users in the list, edit the list, etc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I think we can merge and iterate. We are probably not going to release for a few days so we could add this slowly to main.elk.zone without doing much buzz
Would you resolve the conflicts @boehs? Then let's merge :) |
oops, that merge went terribly. One second |
should be good once checks pass |
Awesome, thanks! A solid first step 🙌🏼 |
#372