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

proposal: add simple users table under charts in admin dashboard #370

Closed
lambtron opened this issue Jul 13, 2023 · 2 comments · Fixed by #396
Closed

proposal: add simple users table under charts in admin dashboard #370

lambtron opened this issue Jul 13, 2023 · 2 comments · Fixed by #396
Labels
enhancement New feature or request v1

Comments

@lambtron
Copy link
Collaborator

  • can we add a users table beneath the charts with some fake users? use case would be the Owner can see/edit/whatever on the SaaS users (see the gif too)

(as example)

image

@lambtron lambtron added the enhancement New feature or request label Jul 13, 2023
@iuioiua
Copy link
Contributor

iuioiua commented Jul 18, 2023

In conjunction with #369, we can do one of the following:

  1. Have both sections on the same page, /dashboard.
  2. Separate both sections into different pages, /dashboard/stats and /dashboard/users, and navigate using tabs like this.

WDYT?

@iuioiua iuioiua added the v1 label Jul 18, 2023
@lambtron
Copy link
Collaborator Author

lets have them both on the same page for now. can we do /admin instead of /dashboard and make hide the route behind an auth? thx!

iuioiua added a commit that referenced this issue Jul 27, 2023
This change also:
* Makes the site text size slightly smaller. Previously, it seemed too
big.
* Adds a `<TabsBar />` component.

Closes #370

![localhost_8000_dashboard_users
(2)](https://github.com/denoland/saaskit/assets/29347852/7f26c0bf-bd4c-4749-b5ae-d921497c0a58)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants