switchfeat icon indicating copy to clipboard operation
switchfeat copied to clipboard

feat(ui): design dashboard

Open dev-bre opened this issue 2 years ago • 20 comments

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.

dev-bre avatar Jul 21 '23 20:07 dev-bre

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.

Zeeshnhmd avatar Jul 22 '23 18:07 Zeeshnhmd

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?

dev-bre avatar Jul 22 '23 19:07 dev-bre

Yes, you can assign this to me.

Zeeshnhmd avatar Jul 22 '23 19:07 Zeeshnhmd

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

dev-bre avatar Jul 31 '23 13:07 dev-bre

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

dev-bre avatar Aug 04 '23 08:08 dev-bre

You'll get it by Sunday EOD

Zeeshnhmd avatar Aug 04 '23 08:08 Zeeshnhmd

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

dev-bre avatar Aug 04 '23 09:08 dev-bre

Any update on this one @Zeeshnhmd?

dev-bre avatar Aug 10 '23 20:08 dev-bre

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

Zeeshnhmd avatar Aug 10 '23 20:08 Zeeshnhmd

Thanks a lot.

dev-bre avatar Aug 10 '23 20:08 dev-bre

@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.

Zeeshnhmd avatar Aug 12 '23 08:08 Zeeshnhmd

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.

dev-bre avatar Aug 12 '23 09:08 dev-bre

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

Zeeshnhmd avatar Aug 12 '23 09:08 Zeeshnhmd

root folder from another terminal window.

On Sat, 12 Aug 2023 at 10:23, Zeeshan Ahmad @.***> wrote:

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

— Reply to this email directly, view it on GitHub https://github.com/switchfeat-com/switchfeat/issues/30#issuecomment-1675810862, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAG5GAHXVFLZOKKZSN476YTXU5DSJANCNFSM6AAAAAA2TJJLTM . You are receiving this because you were mentioned.Message ID: @.***>

dev-bre avatar Aug 12 '23 09:08 dev-bre

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

Zeeshnhmd avatar Aug 12 '23 09:08 Zeeshnhmd

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.

dev-bre avatar Aug 12 '23 09:08 dev-bre

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

image

Zeeshnhmd avatar Aug 12 '23 19:08 Zeeshnhmd

Looks really good @Zeeshnhmd thanks for the effort.

dev-bre avatar Aug 13 '23 05:08 dev-bre

Ok then I will raise a PR for this.

Zeeshnhmd avatar Aug 13 '23 07:08 Zeeshnhmd

PR raised please have a look.

Zeeshnhmd avatar Aug 13 '23 08:08 Zeeshnhmd