[PR #299] [MERGED] fix: default sorting on tables #892

New Issue

OVERLORD · 2025-10-08T00:19:10+03:00

OVERLORD commented

2025-10-08 00:19:10 +03:00

📋 Pull Request Information

Original PR: https://github.com/pocket-id/pocket-id/pull/299
Author: @kmendell
Created: 3/5/2025
Status: ✅ Merged
Merged: 3/6/2025
Merged by: @stonith404

Base: main ← Head: sorting-fixes

📝 Commits (5)

0c15413 fix: default sorting on tables
b5213e3 Merge branch 'main' into sorting-fixes
1957865 fix: rely on server side for sorting
f8e8e3d fix: remove url params from page server sorting logic
a7b293f cleanup default sorting

📊 Changes

7 files changed (+88 additions, -10 deletions)

View changed files

📝 frontend/src/routes/settings/admin/oidc-clients/+page.server.ts (+16 -1)
📝 frontend/src/routes/settings/admin/oidc-clients/oidc-client-list.svelte (+13 -2)
📝 frontend/src/routes/settings/admin/user-groups/+page.server.ts (+15 -1)
📝 frontend/src/routes/settings/admin/user-groups/user-group-list.svelte (+8 -1)
📝 frontend/src/routes/settings/admin/user-groups/user-selection.svelte (+12 -3)
📝 frontend/src/routes/settings/admin/users/+page.server.ts (+15 -1)
📝 frontend/src/routes/settings/admin/users/user-list.svelte (+9 -1)

📄 Description

Fixes: https://github.com/pocket-id/pocket-id/issues/298

_{🔄 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/pocket-id/pocket-id/pull/299 **Author:** [@kmendell](https://github.com/kmendell) **Created:** 3/5/2025 **Status:** ✅ Merged **Merged:** 3/6/2025 **Merged by:** [@stonith404](https://github.com/stonith404) **Base:** `main` ← **Head:** `sorting-fixes` --- ### 📝 Commits (5) - [`0c15413`](https://github.com/pocket-id/pocket-id/commit/0c15413b605bcc91f2fbc09b5851b63ebb07df7d) fix: default sorting on tables - [`b5213e3`](https://github.com/pocket-id/pocket-id/commit/b5213e3102741a2c2f7a22e4c7d05a82a49c69fa) Merge branch 'main' into sorting-fixes - [`1957865`](https://github.com/pocket-id/pocket-id/commit/195786564b57b6fc56cd6ee14a41b7e78f5f15ac) fix: rely on server side for sorting - [`f8e8e3d`](https://github.com/pocket-id/pocket-id/commit/f8e8e3d3b2dcc86118dd15d215620e929409a78b) fix: remove url params from page server sorting logic - [`a7b293f`](https://github.com/pocket-id/pocket-id/commit/a7b293f64372419443254f4808e232d69cbbc1e9) cleanup default sorting ### 📊 Changes **7 files changed** (+88 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `frontend/src/routes/settings/admin/oidc-clients/+page.server.ts` (+16 -1) 📝 `frontend/src/routes/settings/admin/oidc-clients/oidc-client-list.svelte` (+13 -2) 📝 `frontend/src/routes/settings/admin/user-groups/+page.server.ts` (+15 -1) 📝 `frontend/src/routes/settings/admin/user-groups/user-group-list.svelte` (+8 -1) 📝 `frontend/src/routes/settings/admin/user-groups/user-selection.svelte` (+12 -3) 📝 `frontend/src/routes/settings/admin/users/+page.server.ts` (+15 -1) 📝 `frontend/src/routes/settings/admin/users/user-list.svelte` (+9 -1) </details> ### 📄 Description Fixes: https://github.com/pocket-id/pocket-id/issues/298 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

OVERLORD added the pull-request label 2025-10-08 00:19:10 +03:00

OVERLORD closed this issue

2025-10-08 00:19:12 +03:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/pocket-id-pocket-id-1#892