Repository navigation
[Due for payment 2026-10-07] Payment issue for E/App PR#94362 #102705
Description
Activity
melvin-bot commented
on Sep 30, 2026 melvin-botboton Sep 30, 2026 – with Melvin BotAuthorMore actions@huult The PR you reviewed has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the Contributor+ Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]. If no checklist is needed for this issue, you can click: [no checklist button]
Contributor+ Checklist:
-
[Contributor] The offending PR and associated issue have been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
It's a new feature
-
[Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion: N/A
-
[Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.
Regression Test Proposal
Precondition:
Test:
- Open the search router (Cmd+K).
- Search a term matching chats across multiple tiers (your primary workspace + DMs + other workspaces). It's most likely that in order to hit all tiers you may need to run multiple different search terms that would actually return different kinds of reports.
- Verify results display in the proper order: self DM, then your primary workspace chats (including rooms like #admins), then same-domain DMs, then other-workspace chats, then other-domain DMs.
- Verify the list updates to that order once the search returns.
Do we agree 👍 or 👎
-
This ticket was worked on before the price was reduced to $175, so we should keep it at $250.
- addedDailyKSv2KSv2Awaiting PaymentAuto-added when associated PR is deployed to productionAuto-added when associated PR is deployed to productionand removedWeeklyKSv2KSv2
on Oct 6, 2026 Triggered auto assignment to @mallenexpensify (
Awaiting Payment)Payment Summary
- Reviewer: @huult owed $175 via NewDot
BugZero Checklist (@mallenexpensify)
- I have confirmed assignees, roles, and Upwork contracts look correct
- I have paid out Upwork contracts / manual NewDot requests
- [BugZero Assignee] I have created a GH issue for creating/updating the regression test once above steps have been agreed upon
Payment Summary
Contributor+: @huult due $250 via NewDot
Issue created to compensate the Contributor+ member for their work on #94362
E/E issue linked to the PR - no internal issue.
Contributor+ member who reviewed the PR - @huult.
Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.
Issue Owner
Current Issue Owner: @mallenexpensify