Skip to content

fix(author): order author list by createdAt instead of id#257

Merged
volarname merged 1 commit into
mainfrom
fix/author-list-sort-by-created-at
Jun 30, 2026
Merged

fix(author): order author list by createdAt instead of id#257
volarname merged 1 commit into
mainfrom
fix/author-list-sort-by-created-at

Conversation

@volarname

Copy link
Copy Markdown
Contributor

fetchList hard-set pagination.sortBy to 'id', overriding the ordering dropdown (whose default variant already sorts by createdAt) and forcing the list into UUID order, which buried the newest authors. Default to createdAt desc, respect explicit createdAt asc/desc picks, and keep relevance ordering for fulltext search.

@volarname volarname force-pushed the fix/author-list-sort-by-created-at branch from c879e18 to d727d97 Compare June 29, 2026 16:05
Author ids are UUIDs (DocId), so sorting by id produced an arbitrary
order. Both the author list (useAuthorListActions) and the author
select autocomplete (useAuthorSelectActions) hard-set pagination.sortBy
to 'id' on non-text fetches; default them to createdAt so the newest
authors show first, while keeping relevance ordering for fulltext search.
@volarname volarname force-pushed the fix/author-list-sort-by-created-at branch from d727d97 to 25e4f5c Compare June 29, 2026 16:22
@volarname volarname merged commit 963db35 into main Jun 30, 2026
2 checks passed
@volarname volarname deleted the fix/author-list-sort-by-created-at branch June 30, 2026 06:38
volarname pushed a commit that referenced this pull request Jul 1, 2026
Conflict in author/composables/authorActions.ts (moved to src/domains on
this branch): kept the new labs-API version. Preserved main's PR #257
intent (order author lists by createdAt): AuthorDatatable.vue now
usePagination('createdAt') instead of SORT_BY_ID. Author select
autocomplete sort stays on common-admin default (pagination built there,
not controllable from admin-dam).
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