Skip to content

Revert "feat: sort users and projects by last activity by default"#3108

Merged
HarshMN2345 merged 1 commit into
mainfrom
revert-3103-feat/sort-by-accessed-at
Jul 6, 2026
Merged

Revert "feat: sort users and projects by last activity by default"#3108
HarshMN2345 merged 1 commit into
mainfrom
revert-3103-feat/sort-by-accessed-at

Conversation

@HarshMN2345

Copy link
Copy Markdown
Member

Reverts #3103

@HarshMN2345 HarshMN2345 merged commit 1beea5c into main Jul 6, 2026
3 of 4 checks passed
@HarshMN2345 HarshMN2345 deleted the revert-3103-feat/sort-by-accessed-at branch July 6, 2026 11:36
@greptile-apps

greptile-apps Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR reverts #3103, which had introduced accessedAt-based descending sort for the projects and users list pages. Both queries are restored to Query.orderDesc(''), which uses Appwrite's default sort order.

  • organization-[organization]/+page.ts: Project listing sort reverts from Query.orderDesc('accessedAt') to Query.orderDesc('').
  • project-[region]-[project]/auth/+page.ts: User listing sort reverts from Query.orderDesc('accessedAt') to Query.orderDesc('').

Confidence Score: 5/5

Safe to merge — the change is a mechanical two-line revert with no side effects beyond restoring the previous sort order.

Both files simply swap 'accessedAt' back to an empty string in Query.orderDesc. No logic, auth, or data-mutation paths are affected.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/organization-[organization]/+page.ts Reverts project listing sort order from Query.orderDesc('accessedAt') back to Query.orderDesc('') (default sort)
src/routes/(console)/project-[region]-[project]/auth/+page.ts Reverts user listing sort order from Query.orderDesc('accessedAt') back to Query.orderDesc('') (default sort)

Reviews (1): Last reviewed commit: "Revert "feat: sort users and projects by..." | Re-trigger Greptile

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.

2 participants