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(ui): design dashboard #30

Open
dev-bre opened this issue Jul 21, 2023 · 20 comments
Open

feat(ui): design dashboard #30

dev-bre opened this issue Jul 21, 2023 · 20 comments
Assignees

Comments

@dev-bre
Copy link
Contributor

dev-bre commented Jul 21, 2023

The first iteration of the dashboard page should fairly simple but eye catching.
I would think having cards showing some stats against how many flags / segments / rules have been created. how many flags are enabled.

Once we have the basic stuff, we might expand with some more in depth analytics, charts, etc.

@Zeeshnhmd
Copy link
Contributor

Hey @dev-bre I would love to work on this issue. But before starting I have some questions to ask.

  1. Do you have any design in mind? If not please provide a design so that I can give a pixel-perfect design.
  2. Can I use module.scss for the following design?

Thank you.

@dev-bre
Copy link
Contributor Author

dev-bre commented Jul 22, 2023

Hi, didn't think of a design yet, but for the first iteration, having like 3 cards showing some stats in terms of:
Number flags and the percentage of active flags
Number of segments
Number of rules

And maybe a History of actions like an audit list
Added new flag on (date)
Removed flag on (date)
...

All these data can be hardcoded for the scope of this task.

I was looking at this dashboard design and looks pretty cool:
https://justboil.me/tailwind-admin-templates/free-react-dashboard/

For now I would focus on the style of the 3 cards. Maybe also the style of that sidebar, the colour contrast there looks amazing.

Please use Tailwind CSS and not any other styling logic , I would like to keep consistency.

Sounds good, for me to assign this to you?

@Zeeshnhmd
Copy link
Contributor

Yes, you can assign this to me.

@dev-bre
Copy link
Contributor Author

dev-bre commented Jul 31, 2023

@Zeeshnhmd do you have an idea when you can start working on it? if you still want to?

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 4, 2023

@Zeeshnhmd can you please let me know if you are still interested in this task? Otherwise I can reassign it?

@Zeeshnhmd
Copy link
Contributor

You'll get it by Sunday EOD

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 4, 2023

no rush at all. just wanted to make sure you were still up for it! all good thanks a lot.

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 10, 2023

Any update on this one @Zeeshnhmd?

@Zeeshnhmd
Copy link
Contributor

Hey, slipped my mind I'll try to deliver this task this weekend.

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 10, 2023

Thanks a lot.

@Zeeshnhmd
Copy link
Contributor

Zeeshnhmd commented Aug 12, 2023

@dev-bre Dude, there is something wrong with your tailwind configuration it requires restarting the app whenever I am changing or applying a class to elements.

How can we fix this?

we can connect over Discord to see the issue.

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 12, 2023

there is an additional npm script you can use in the package.json which reloads the TW changes:

"dev:start-tw:watch": "cd ./packages/ui && npm run build:tw:watch"

just run:

npm run dev:start-tw:watch

and you will good to go. I will update the readme for this.

@Zeeshnhmd
Copy link
Contributor

Do i need to run this inside the UI folder or root folder?

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 12, 2023 via email

@Zeeshnhmd
Copy link
Contributor

No its not working.
Steps I have followed:-

  1. Add this script in root package.json "dev:start-tw:watch": "cd ./packages/ui && npm run build:tw:watch".
  2. And then run this script npm run dev:start-tw:watch in the root folder in another terminal

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 12, 2023

the script is already in the root of package.json, you dont have to add it. Please make sure to sync with the latest main branch.

@Zeeshnhmd
Copy link
Contributor

Have a look at the UI let me know if you need any changes.

image

@dev-bre
Copy link
Contributor Author

dev-bre commented Aug 13, 2023

Looks really good @Zeeshnhmd thanks for the effort.

@Zeeshnhmd
Copy link
Contributor

Ok then I will raise a PR for this.

@Zeeshnhmd
Copy link
Contributor

PR raised please have a look.

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

No branches or pull requests

2 participants