[PR #219] [CLOSED] Fix event sort order #221

New Issue

giteasync · 2025-10-09T17:00:21-05:00

giteasync commented

2025-10-09 17:00:21 -05:00

📋 Pull Request Information

Original PR: https://github.com/Cleanuparr/Cleanuparr/pull/219
Author: @GeekJosh
Created: 7/2/2025
Status: ❌ Closed

Base: dev ← Head: patch-1

📝 Commits (1)

565b1a3 fixed event sort order

📊 Changes

1 file changed (+1 additions, -5 deletions)

View changed files

📝 code/backend/Cleanuparr.Api/Controllers/EventsController.cs (+1 -5)

📄 Description

In my opinion, the current behaviour is confusing. Each page of events is sorted in ascending order by the timestamp, but the pages are in descending order by timestamp. This feels off to me, so I'm putting forward this PR to keep events in descending order, which feels more natural.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Cleanuparr/Cleanuparr/pull/219 **Author:** [@GeekJosh](https://github.com/GeekJosh) **Created:** 7/2/2025 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`565b1a3`](https://github.com/Cleanuparr/Cleanuparr/commit/565b1a3755703c22d132adb9a4f0b07b3a559f3b) fixed event sort order ### 📊 Changes **1 file changed** (+1 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `code/backend/Cleanuparr.Api/Controllers/EventsController.cs` (+1 -5) </details> ### 📄 Description In my opinion, the current behaviour is confusing. Each page of events is sorted in ascending order by the timestamp, but the pages are in descending order by timestamp. This feels off to me, so I'm putting forward this PR to keep events in descending order, which feels more natural. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

giteasync added the

pull-request

label 2025-10-09 17:00:21 -05:00

giteasync closed this issue

2025-10-09 17:00:22 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/Cleanuparr#221