Skip to content

[Due for payment 2026-09-09] [SelectionList ListItem] PR 2 — SelectableListItem adapter + BaseSelectListItem + TravelDomainListItem + CardListItem #98359

Closed

Description

@MelvinBot

Part of the SelectionList ListItem — Composition over Configuration tracking effort. See PR split in this comment.

PR 2 — SelectableListItem adapter + BaseSelectListItem + TravelDomainListItem + CardListItem

  • Migrates: BaseSelectListItem / SingleSelectListItem, TravelDomainListItem, CardListItem. SingleSelectWithAvatarListItem.
  • Components needed: SelectableListItem adapter (checkbox/radio via ListSelectionButton, role derived from canSelectMultiple, composed on ListItem.Pressable); TextColumn.
  • Deletes: components/ListCheckbox.tsx, components/ListRadioButton.tsx — pure role-fixing wrappers; callers pass role={CONST.ROLE.CHECKBOX} / {CONST.ROLE.RADIO} on ListSelectionButton directly.
  • Implementation details:
    • Multiline: styles.preWrap + inline {paddingLeft}; muted: styles.colorMuted; subtitle maxWidth via style.
Issue OwnerCurrent Issue Owner: @mallenexpensify

Activity

  1. roryabraham commented on Aug 13, 2026

    @roryabraham
    Contributor

    Awaiting PR

  2. melvin-bot commented on Aug 17, 2026

    @melvin-bot

    @jmusial, @roryabraham, @truph01 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

  3. melvin-bot commented on Aug 19, 2026

    @melvin-bot

    @jmusial, @roryabraham, @truph01 Eep! 4 days overdue now. Issues have feelings too...

  4. roryabraham commented on Aug 19, 2026

    @roryabraham
    Contributor

    No update. Still reviewing PR 1

  5. roryabraham commented on Aug 24, 2026

    @roryabraham
    Contributor

    I approved PR 1 and it's awaiting C+ review.

    Looks like @jmusial has a draft PR for this

  6. 9 remaining items

  7. roryabraham commented on Sep 7, 2026

    @roryabraham
    Contributor

    Status update — 2026-09-07

    Complete. PR #99177 merged on Sep 1 and was deployed to production in version 9.4.68-1. The issue is in the 7-day regression/payment flow, with payment currently scheduled for Sep 9 if no regression occurs.

  8. melvin-bot commented on Sep 7, 2026

    @melvin-bot

    @truph01 Huh... This is 4 days overdue. Who can take care of this?

  9. melvin-bot commented on Sep 8, 2026

    @melvin-bot

    Triggered auto assignment to @mallenexpensify (Awaiting Payment)

  10. melvin-bot commented on Sep 8, 2026

    @melvin-bot

    Payment Summary

    Resolving PRs:

    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
  11. mallenexpensify commented on Sep 10, 2026

    @mallenexpensify
    Contributor

    @truph01 is this project paid?

  12. truph01 commented on Sep 11, 2026

    @truph01
    Contributor

    @mallenexpensify This will be paid via #97971. You can close it.

  13. added
    Skip PaymentSkips all the payment process steps for an issue. Use it for fixed-price projects.
    on Sep 12, 2026
  14. MelvinBot commented on Sep 16, 2026

    @MelvinBot
    ContributorAuthor

    Status update — 2026-09-16

    Complete. SelectableListItem adapter + BaseSelectListItem + TravelDomainListItem + CardListItem merged Sep 1 and deployed. This issue was closed Sep 12.

    Bookkeeping only: it still carries Awaiting Payment alongside Skip Payment. Payment for this project runs through the parent tracking issue at $1,000 total, so the Awaiting Payment label can come off.

    Tracking: [$1000] [Tracking] SelectionList ListItem - Composition over Configuration

  15. MelvinBot commented on Oct 6, 2026

    @MelvinBot
    ContributorAuthor

    Status update — 2026-10-06

    Complete. No change since the last update. SelectableListItem adapter + BaseSelectListItem + TravelDomainListItem + CardListItem merged Sep 1 and is in production. This issue closed Sep 12.

    Bookkeeping: this issue still has Awaiting Payment next to Skip Payment. Payment runs once on the tracking issue, so remove Awaiting Payment.

    Tracking: [$1000] [Tracking] SelectionList ListItem - Composition over Configuration


    view run

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 productionDailyKSv2Skip PaymentSkips all the payment process steps for an issue. Use it for fixed-price projects.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions