Skip to content

[Due for payment 2026-10-07] Payment issue for E/App PR#94362 #102705

Description

@melvin-bot

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 OwnerCurrent Issue Owner: @mallenexpensify

Activity

  1. melvin-bot commented on Sep 30, 2026

    @melvin-bot
    Author

    @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]

  2. huult commented on Oct 1, 2026

    @huult
    Contributor

    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:

    1. Open the search router (Cmd+K).
    2. 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.
    3. 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.
    4. Verify the list updates to that order once the search returns.

    Do we agree 👍 or 👎

  3. huult commented on Oct 1, 2026

    @huult
    Contributor

    This ticket was worked on before the price was reduced to $175, so we should keep it at $250.

  4. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Oct 6, 2026
  5. melvin-bot commented on Oct 6, 2026

    @melvin-bot
    Author

    Triggered auto assignment to @mallenexpensify (Awaiting Payment)

  6. melvin-bot commented on Oct 6, 2026

    @melvin-bot
    Author

    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
  7. mallenexpensify commented on Oct 8, 2026

    @mallenexpensify
    Contributor

    Payment Summary

    Contributor+: @huult due $250 via NewDot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2Task

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions