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

Create Dashboard Site #32

Closed
SilvanBaach opened this issue Apr 1, 2023 · 6 comments · Fixed by #35
Closed

Create Dashboard Site #32

SilvanBaach opened this issue Apr 1, 2023 · 6 comments · Fixed by #35
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@SilvanBaach
Copy link
Owner

SilvanBaach commented Apr 1, 2023

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
Implement the Dashboard site like in the sketch. There should be a left side nav bar which should be extendable. The rest of the page should be an empty

. Implement some snippet loading logic so that when you click on a nav bar icon the empty section gets replaced with the actual ejs content from that page. If there are any js libs which make handling this single page application easier propse it to @SilvanBaach .

After successfull login the user should be redirectet to the dashboard page. Create an express route for this. On Default home.ejs should be loaded into the

.

Describe alternatives you've considered

Additional context
Sketch:
dashboard

@SilvanBaach SilvanBaach added the enhancement New feature or request label Apr 1, 2023
@SilvanBaach SilvanBaach added this to the Feature: Dashboard milestone Apr 1, 2023
@SilvanBaach
Copy link
Owner Author

@BadrOutiti

@BadrOutiti
Copy link
Collaborator

@jaypi95 Created a first draft, please review it!

@jaypi95
Copy link
Collaborator

jaypi95 commented Apr 19, 2023

LGTM. I've converted the draft PR to a merging ready PR. I've asked @SilvanBaach for an additional review on the PR directly.

@jaypi95
Copy link
Collaborator

jaypi95 commented Apr 19, 2023

Talked to Silvan on another channel.
According to him the following is missing:

  • Animations are not working as intended
  • Logout button is missing
  • Route in server.js is incorrect
  • Icon is in wrong position

PR has been converted back to a draft

BadrOutiti pushed a commit that referenced this issue Apr 19, 2023
BadrOutiti pushed a commit that referenced this issue Apr 19, 2023
@BadrOutiti
Copy link
Collaborator

Talked to Silvan on another channel. According to him the following is missing:

  • Animations are not working as intended
  • Logout button is missing
  • Route in server.js is incorrect
  • Icon is in wrong position

PR has been converted back to a draft

-Animation works now
-Logout Button was added (Does not do anything right now)
-Route was added
-Icon was changed after speaking with @SilvanBaach

Please Review it again!

@jaypi95
Copy link
Collaborator

jaypi95 commented Apr 19, 2023

Can't run it since I'm at work but it looks like everything was done as requested. I'll ask Silvan again for another review.

BadrOutiti pushed a commit that referenced this issue Apr 19, 2023
…hat we use the styles of the genetix brandbook[#32]
ArmandoShala added a commit that referenced this issue Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants