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(dashboard): render dashboard page and repo cards #796

Merged
merged 32 commits into from
May 30, 2024

Conversation

KellyMerrick
Copy link
Contributor

@KellyMerrick KellyMerrick commented Apr 26, 2024

Dashboard view with multiple repo cards:

theme-dark
image

theme-light
image

Dashboard with no repos:
image

No dashboards:
image

@KellyMerrick
Copy link
Contributor Author

Need to add comments still.

@KellyMerrick KellyMerrick changed the title feat(dashboard): renders unformatted dashboard and repo cards feat(dashboard): render dashboard page and repo cards May 20, 2024
@KellyMerrick KellyMerrick marked this pull request as ready for review May 24, 2024 18:42
@KellyMerrick KellyMerrick requested a review from a team as a code owner May 24, 2024 18:42
@plyr4
Copy link
Contributor

plyr4 commented May 28, 2024

i really like the 'no dashboards' view. friendly and helpful. need more of that on the rest of the site!

src/scss/_dashboards.scss Outdated Show resolved Hide resolved
@plyr4
Copy link
Contributor

plyr4 commented May 28, 2024

overall, epic. excited to see where this goes.
couple tiny comments

Copy link
Contributor

@plyr4 plyr4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sweet! thanks for the updates too

@KellyMerrick KellyMerrick merged commit d0dd9e9 into main May 30, 2024
13 checks passed
@KellyMerrick KellyMerrick deleted the feat/dashboard branch May 30, 2024 14:42
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.

4 participants