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

Give option to sort or filter by Created timestamp #2293

Merged
merged 11 commits into from
Aug 31, 2022

Conversation

mattseddon
Copy link
Member

@mattseddon mattseddon commented Aug 30, 2022

This PR gives users the ability to sort/filter by the created column. Follow up from #2239 (comment).

Demo

https://user-images.githubusercontent.com/37993418/187358368-d7bdd74d-123d-42ec-8f0b-60a49f4d0754.mov

Screen.Recording.2022-08-31.at.10.15.29.am.mov

@mattseddon mattseddon added the product PR that affects product label Aug 30, 2022
@mattseddon mattseddon self-assigned this Aug 30, 2022
label: columnLike.label,
value: columnLike
})),
columnLikes.map(columnLike => ({
Copy link
Member Author

Choose a reason for hiding this comment

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

[F] Sorting takes care of itself because the timestamp is an ISO string.

@mattseddon mattseddon marked this pull request as ready for review August 30, 2022 10:33
Copy link
Contributor

@julieg18 julieg18 left a comment

Choose a reason for hiding this comment

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

Since timestamps are sortable now, do we want to update their context menu to include sorting options:

image

@mattseddon
Copy link
Member Author

Thanks, @julieg18 I'll fix everything.

@mattseddon mattseddon requested a review from julieg18 August 31, 2022 00:17
@codeclimate
Copy link

codeclimate bot commented Aug 31, 2022

Code Climate has analyzed commit 362cf8f and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 1

The test coverage on the diff in this pull request is 83.6% (85% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.

@mattseddon mattseddon merged commit 6eade74 into main Aug 31, 2022
@mattseddon mattseddon deleted the sort-filter-by-created branch August 31, 2022 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product PR that affects product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants