Skip to content

fix: update evaluator applicant ordering by submission time#106

Merged
ErpingS merged 4 commits intodevfrom
erping/evaluator-applicants-ordering
Apr 8, 2026
Merged

fix: update evaluator applicant ordering by submission time#106
ErpingS merged 4 commits intodevfrom
erping/evaluator-applicants-ordering

Conversation

@ErpingS
Copy link
Copy Markdown
Contributor

@ErpingS ErpingS commented Feb 24, 2026

Reason

Applicant ordering was previously by firebase id, now it the ordering is by submission time (oldest to newest)

Explanation

Other considerations

naijwu and others added 4 commits February 16, 2026 16:09
push evaluator changes to prod
* hotfix(evaluator): add additional response field for cmd-f
we probably want to move these values into firestore eventually

* Evaluator settings fixes (#101)

* fix: evalutor settings modal overflow

* fix: update step increment and error msg spacing

* minor: rename eval settings prop type

---------

Co-authored-by: Jae Wu <jaewuchun@gmail.com>

---------

Co-authored-by: Erping <111095333+ErpingS@users.noreply.github.com>
@ErpingS ErpingS requested a review from tdanielles February 24, 2026 02:21
@github-actions
Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit df045d5):

https://dev-nwplus-admin--pr106-erping-evaluator-app-vgmsnthi.web.app

(expires Fri, 06 Mar 2026 02:22:36 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c290bf9e2fac0401389f751f415cce4267517b51

@ErpingS ErpingS merged commit 4f6951b into dev Apr 8, 2026
5 checks passed
@ErpingS ErpingS deleted the erping/evaluator-applicants-ordering branch April 8, 2026 02:54
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.

3 participants