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-HNG-202): user dashboard overview page layout for Team-Starlight #717

Merged
merged 6 commits into from
Jul 31, 2024
Merged

(feat-HNG-202): user dashboard overview page layout for Team-Starlight #717

merged 6 commits into from
Jul 31, 2024

Conversation

elitenoire
Copy link
Contributor

@elitenoire elitenoire commented Jul 29, 2024

Description

Closes #202

Changes proposed

What were you told to do?

User Dashboard overview page - key metrics, graph, recent sales

What did you do?

  • Provided the user dashboard layout
  • Added calendar and menu links to different sections

Check List (Check all the applicable boxes)

🚨Please review the contribution guideline for this repository.

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title and description of the PR is clear and explains the approach.
  • I am making a pull request against the dev branch (left side).
  • My commit messages styles matches our requested structure.
  • My code additions will fail neither code linting checks nor unit test.
  • I am only making changes to files I was requested to.

Screenshots/Videos

Desktop
desktop
Tablet
tablet

@elitenoire
Copy link
Contributor Author

elitenoire commented Jul 29, 2024

Not ready for merge yet; still awaiting recent sales and metrics component from PR #718 for integration.
Needs PR #718 merged first.

@kleenpulse
Copy link
Contributor

this PR fixes the navbar as well?

@elitenoire
Copy link
Contributor Author

this PR fixes the navbar as well?

Nooo, left the navbar as it is. Perhaps that will be done in another PR

@elitenoire elitenoire marked this pull request as ready for review July 29, 2024 09:09
@elitenoire
Copy link
Contributor Author

Maybe I should open multiple PRs to reduce the number of files?

@elitenoire
Copy link
Contributor Author

Please all fixes have been made. Awaiting to be Merged, please @PrudentBird

@kleenpulse
Copy link
Contributor

Fix building errors

@elitenoire
Copy link
Contributor Author

@kleenpulse I have fixed the build errors.

kleenpulse
kleenpulse previously approved these changes Jul 30, 2024
Maiqel1
Maiqel1 previously approved these changes Jul 30, 2024
@elitenoire elitenoire dismissed stale reviews from Maiqel1 and kleenpulse via 7b0bac7 July 31, 2024 01:33
@elitenoire
Copy link
Contributor Author

@kleenpulse @DestinedCodes @incredible-phoenix246 all fixes made. Please help approve

@elitenoire elitenoire requested a review from kleenpulse July 31, 2024 01:48
@incredible-phoenix246 incredible-phoenix246 merged commit 14bd53c into hngprojects:dev Jul 31, 2024
3 checks passed
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.

[Feature]: [FEAT] User Dashboard Page
4 participants