The GitHub Actions job "Required Checks" on 
texera.git/fix/admin-user-sort-direction has succeeded.
Run started by GitHub user xuang7 (triggered by xuang7).

Head commit for run:
0635042163b3fd1d254cfcc8952d83e15f3cef22 / Xinyuan Lin <[email protected]>
fix(frontend): sort the admin user ID column ascending under the ascend caret

nz-table applies an NzTableSortFn's result as-is for 'ascend' and negates it
for 'descend', so sortByID's reversed operands rendered the highest uid first
under the up caret. The reversal was not a "newest accounts first" default: no
column sets nzSortOrder, so the comparator only runs once a header is clicked,
and Account Creation Time, the column that actually encodes recency, already
sorts oldest-first.

Extend the rendered-header table with the ID column so it pins the row order
and the active caret in both directions, and supply the rows in an order that
matches no column's ascending or descending expectation, so no leg can pass
against a table that does not sort. Also note why sortByActive's
active-before-inactive ranking is its ascending order rather than a reversal.

Report URL: https://github.com/apache/texera/actions/runs/35292914891

With regards,
GitHub Actions via GitBox

Reply via email to