[PR #1548] [MERGED] fix: admin leaderboard query sorting #1708

Closed
opened 2026-02-26 20:30:43 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/documenso/documenso/pull/1548
Author: @ephraimduncan
Created: 12/19/2024
Status: Merged
Merged: 1/28/2025
Merged by: @Mythie

Base: mainHead: fix/admin-leaderboard-query-sorting


📝 Commits (8)

  • 0e08825 chore: staging
  • a44e4a0 chore: use kysely
  • 838dda3 chore: wip
  • 0c2c092 chore: sorting indicators
  • 5d760e0 Merge branch 'main' into fix/admin-leaderboard-query-sorting
  • 5596344 chore: trigger redeployment
  • f478f91 fix: preserve search params during leaderboard sorting
  • ac7a91e Merge branch 'main' into fix/admin-leaderboard-query-sorting

📊 Changes

2 files changed (+102 additions, -89 deletions)

View changed files

📝 apps/web/src/app/(dashboard)/admin/leaderboard/data-table-leaderboard.tsx (+32 -5)
📝 packages/lib/server-only/admin/get-signing-volume.ts (+70 -84)

📄 Description


name: Pull Request
about: Submit changes to the project for review and inclusion

Description

Changes Made

  • Change 1
  • Change 2
  • ...

Testing Performed

  • Tested feature X in scenario Y.
  • Ran unit tests for component Z.
  • Tested on browsers A, B, and C.
  • ...

Checklist

  • I have tested these changes locally and they work as expected.
  • I have added/updated tests that prove the effectiveness of these changes.
  • I have updated the documentation to reflect these changes, if applicable.
  • I have followed the project's coding style guidelines.
  • I have addressed the code review feedback from the previous submission, if applicable.

Additional Notes


🔄 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/documenso/documenso/pull/1548 **Author:** [@ephraimduncan](https://github.com/ephraimduncan) **Created:** 12/19/2024 **Status:** ✅ Merged **Merged:** 1/28/2025 **Merged by:** [@Mythie](https://github.com/Mythie) **Base:** `main` ← **Head:** `fix/admin-leaderboard-query-sorting` --- ### 📝 Commits (8) - [`0e08825`](https://github.com/documenso/documenso/commit/0e08825a67c875d1e99c022f4a50575f2600c98c) chore: staging - [`a44e4a0`](https://github.com/documenso/documenso/commit/a44e4a02ce65e8f3cb261d853d5ced928b07e957) chore: use kysely - [`838dda3`](https://github.com/documenso/documenso/commit/838dda35a76ab60081ba180e87616952040cd55f) chore: wip - [`0c2c092`](https://github.com/documenso/documenso/commit/0c2c0923dd03d3ddaf5f1a402fa1b44c7353f88b) chore: sorting indicators - [`5d760e0`](https://github.com/documenso/documenso/commit/5d760e07d32a307f63520051cd142c42b894e808) Merge branch 'main' into fix/admin-leaderboard-query-sorting - [`5596344`](https://github.com/documenso/documenso/commit/55963441b2e24436e1263037ac3bc56d7765d85c) chore: trigger redeployment - [`f478f91`](https://github.com/documenso/documenso/commit/f478f917f3d9ece5f68d71da865cde33a043bfef) fix: preserve search params during leaderboard sorting - [`ac7a91e`](https://github.com/documenso/documenso/commit/ac7a91eeca79981bd2ad4556450fa29a2273c839) Merge branch 'main' into fix/admin-leaderboard-query-sorting ### 📊 Changes **2 files changed** (+102 additions, -89 deletions) <details> <summary>View changed files</summary> 📝 `apps/web/src/app/(dashboard)/admin/leaderboard/data-table-leaderboard.tsx` (+32 -5) 📝 `packages/lib/server-only/admin/get-signing-volume.ts` (+70 -84) </details> ### 📄 Description --- name: Pull Request about: Submit changes to the project for review and inclusion --- ## Description <!--- Describe the changes introduced by this pull request. --> <!--- Explain what problem it solves or what feature/fix it adds. --> ## Related Issue <!--- If this pull request is related to a specific issue, reference it here using #issue_number. --> <!--- For example, "Fixes #123" or "Addresses #456". --> ## Changes Made <!--- Provide a summary of the changes made in this pull request. --> <!--- Include any relevant technical details or architecture changes. --> - Change 1 - Change 2 - ... ## Testing Performed <!--- Describe the testing that you have performed to validate these changes. --> <!--- Include information about test cases, testing environments, and results. --> - Tested feature X in scenario Y. - Ran unit tests for component Z. - Tested on browsers A, B, and C. - ... ## Checklist <!--- Please check the boxes that apply to this pull request. --> <!--- You can add or remove items as needed. --> - [ ] I have tested these changes locally and they work as expected. - [ ] I have added/updated tests that prove the effectiveness of these changes. - [ ] I have updated the documentation to reflect these changes, if applicable. - [ ] I have followed the project's coding style guidelines. - [ ] I have addressed the code review feedback from the previous submission, if applicable. ## Additional Notes <!--- Provide any additional context or notes for the reviewers. --> <!--- This might include details about design decisions, potential concerns, or anything else relevant. --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 20:30:43 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/documenso#1708
No description provided.