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

Make favorite queries/dashboard order by starred at(favorited at) #7351

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

yoshiokatsuneo
Copy link
Contributor

@yoshiokatsuneo yoshiokatsuneo commented Feb 26, 2025

What type of PR is this?

  • Feature

Description

Before this PR, favorite dashboards and queries are ordered by "created_at".
So, older dashboards/queries is always at the bottom of the list that is difficult to reach out.

This PR changes the sorting order by "starred at" (the time when user favorited). If the user want to put specific dashboard on the top of the list, the user can do it by just re-favorite it.

How is this tested?

  • Manually

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

image image

@yoshiokatsuneo yoshiokatsuneo force-pushed the feat/favorites_order_by_starred_at branch 2 times, most recently from 9e612e6 to 0ed92e3 Compare February 26, 2025 17:52
@yoshiokatsuneo yoshiokatsuneo marked this pull request as ready for review February 26, 2025 17:53
@yoshiokatsuneo yoshiokatsuneo force-pushed the feat/favorites_order_by_starred_at branch 2 times, most recently from fd91171 to d4f21db Compare March 3, 2025 05:09
@yoshiokatsuneo yoshiokatsuneo force-pushed the feat/favorites_order_by_starred_at branch from d4f21db to da4775e Compare March 3, 2025 05:17
@yoshiokatsuneo yoshiokatsuneo force-pushed the feat/favorites_order_by_starred_at branch from da4775e to ed90a19 Compare March 3, 2025 05:23
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.

1 participant