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.

Sign up for GitHub

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

Jump to bottom

feat: landing screen for projects #534

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

feat: landing screen for projects #534

wants to merge 28 commits into from

Conversation

cabreraalex
Copy link
Member

@cabreraalex cabreraalex commented Dec 7, 2023 •
edited by Sparkier
Loading

Description

Big change to show a home landing screen for projects with quick insights.

Status & TODO

  • Generate different home charts depending on # of models and metrics. I'm thinking maybe some sort of small multiple bar charts too instead of spider? Spider chart is hard to render consistently for different metric scales...
  • Prevent deletion of home charts in chart view
  • Show chart edit button and titles
  • Allow sort by metrics w/ ascend and descend
  • Fix sizing of elements

Fix ZEN-405

@linear Linear
Copy link

linear bot commented Dec 7, 2023

ZEN-405 create home landing screen for projects

@cabreraalex cabreraalex marked this pull request as draft December 7, 2023 21:23
@Sparkier Sparkier force-pushed the ac-project-home branch 3 times, most recently from 39d4233 to d544242 Compare December 12, 2023 16:05
@Sparkier Sparkier marked this pull request as ready for review December 20, 2023 18:12
@Sparkier Sparkier self-requested a review December 20, 2023 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sparkier Sparkier Awaiting requested review from Sparkier

At least 1 approving review is required to merge this pull request.

Assignees

@cabreraalex cabreraalex

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cabreraalex @Sparkier