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

feat: admin pannel #409

Closed
wants to merge 1 commit into from
Closed

feat: admin pannel #409

wants to merge 1 commit into from

Conversation

deer
Copy link
Contributor

@deer deer commented Aug 2, 2023

closes #73

Hey @iuioiua, this is incredibly rough right now, but you encouraged me to open something so we can iterate on the idea.

With these changes, then I can go to the following routes:

There are currently two major issues:

  • The styling is horrible. I need to get the plugin to accept styles that get applied to the routes it creates.
  • The plugin only works with pentagon right now. So most of the functionality doesn't work! I'm at least glad that I can show the items, otherwise I wouldn't even open this. Ideally the user of the plugin could map their own CRUD functions to types, instead of forcing the plugin user to create (possibly unnecessary/unwanted) pentagon models.

I will get to these enhancements at some point and then post an update here. Hopefully that will get rid of the pentagon and zod imports.

After those are dealt with, I can turn to a better layout (likely some sort of table) and then security / permissions.

@iuioiua
Copy link
Contributor

iuioiua commented Aug 3, 2023

Interested to see what this evolves into!

@iuioiua
Copy link
Contributor

iuioiua commented Aug 18, 2023

Hey Reed, are you cool if we close this PR without merging for now? We can revisit once v1 has landed.

@deer deer closed this Aug 18, 2023
@deer
Copy link
Contributor Author

deer commented Aug 18, 2023

Not at all a problem. Work picked up and I unfortunately had less time to contribute to this than I had hoped.

@iuioiua
Copy link
Contributor

iuioiua commented Aug 18, 2023

All good! Thanks anyway!

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.

proposal: admin dashboard
2 participants