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

Repo list doesn't show ordering/sort default #1051

Closed
Tracked by #1049
codecovdesign opened this issue Jan 16, 2024 · 2 comments
Closed
Tracked by #1049

Repo list doesn't show ordering/sort default #1051

codecovdesign opened this issue Jan 16, 2024 · 2 comments
Assignees
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. good first issue Good for newcomers Polish polish, tweaks, and bug issues that are small iterations and/or not part of larger quarterly epics

Comments

@codecovdesign
Copy link
Contributor

codecovdesign commented Jan 16, 2024

Problem to solve

After sign up, user landing on repo page, there is confusion about the 1) sorting of the repos, and 2) all/acitve/inactive copy. Example see video in 4:45 where user is hoping it's sorted by latest activity (it is sorted by latest activity, though it is not displaying in the column in "last updated")

Additionally, the all/active/inactive is though to be related to GH repo active vs configuration status. Context previously it copy showed "not yet configured" to be more explicit. Active and inactive is carryover copy from the concept of activated and/or deactivating repos.

Solution ideation

  • Show last updated activity time stamp in repo list column (since it is sorted this way by default?)
    • Investigation: what is the sort default? is it true that it's by activity? if yes, let's show the th sort
    • Double check if this table is the refactored table
  • Consider alternative copy in the all/active/inactive to better indicate configuration status
  • Related: Update CTA treatment for not-yet-configured repos #235

all / active / inactive => all / configured / not yet configured

Setup repo => Configured

view designs

@codecovdesign codecovdesign added in discovery The design, product, and specifications require refinement Polish polish, tweaks, and bug issues that are small iterations and/or not part of larger quarterly epics labels Jan 16, 2024
@codecovdesign codecovdesign added Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. good first issue Good for newcomers and removed in discovery The design, product, and specifications require refinement labels Feb 9, 2024
@ajay-sentry
Copy link

@codecov/design Is this a typo? If so, what should this read as?

let's show the th sort

Confirming that the list is already sorted by most recently updated by default

Also, Figma shows "Not Configured" while ticket mentions "Not Yet Configured" for the copy. I went with what's on the figma as it's a little "cleaner" imo. Video attached

Screen.Recording.2024-02-20.at.3.38.32.PM.mov

@ajay-sentry
Copy link

Confirmed this is using the new table

@codecov-hooky codecov-hooky bot closed this as completed Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dev-Ready This means the UX is reviewed and ready for prioritization scheduling. good first issue Good for newcomers Polish polish, tweaks, and bug issues that are small iterations and/or not part of larger quarterly epics
Projects
None yet
Development

No branches or pull requests

3 participants