Skip to content

[$250] Subscription - "Trial:" text is not centered after removing the subtitle #74726

Closed

Description

@lanitochka17

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.2.48-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: #74531
Issue reported by: Applause - Internal Team

Action Performed:

  1. Open Expensify Android app
  2. Create New Account > Create Workspace > Go to subscriptions
  3. Paste the following in console: Onyx.merge('nvp_expensify_billingFundID', 32)

Expected Result:

The subtitle "Add a payment card to continue using all of your favorite features." is removed and the title is centered

Actual Result:

The subtitle "Add a payment card to continue using all of your favorite features." is removed but the title is not centered

Workaround:

Unknown

Platforms:

Select the officially supported platforms where the issue was reproduced:

  • Android: App
  • Android: mWeb Chrome
  • iOS: App
  • iOS: mWeb Safari
  • iOS: mWeb Chrome
  • Windows: Chrome
  • MacOS: Chrome / Safari
  • MacOS: Desktop
Platforms Tested: On which of our officially supported platforms was this issue tested:
  • Android: App
  • Android: mWeb Chrome
  • iOS: App
  • iOS: mWeb Safari
  • iOS: mWeb Chrome
  • Windows: Chrome
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence
Bug6999665_1762793498305.f6966f64-ea2e-44be-9e46-804514431f2d.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021988216255221991275
  • Upwork Job ID: 1988216255221991275
  • Last Price Increase: 2025-11-11
  • Automatic offers:
    • samranahm | Contributor | 109478311
Issue OwnerCurrent Issue Owner: @

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    BugSomething is broken. Auto assigns a BugZero manager.
    on Nov 10, 2025
  2. melvin-bot commented on Nov 10, 2025

    @melvin-bot

    Triggered auto assignment to @VictoriaExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

  3. melvin-bot commented on Nov 10, 2025

    @melvin-bot

    Triggered auto assignment to @AndrewGable (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  4. melvin-bot commented on Nov 10, 2025

    @melvin-bot

    💬 A slack conversation has been started in #expensify-open-source

  5. github-actions commented on Nov 10, 2025

    @github-actions
    Contributor

    👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  6. changed the title [-]Android - Subscription - "Trial:" text is not centered after removing the subtitle[/-] [+]Subscription - "Trial:" text is not centered after removing the subtitle[/+] on Nov 10, 2025
  7. samranahm commented on Nov 10, 2025

    @samranahm
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    Subscription - "Trial:" text is not centered after removing the subtitle

    What is the root cause of that problem?

    In BillingBanner.tsx, the subtitle was always rendered regardless of whether it contained content

    When payment card is added via Onyx.merge('nvp_expensify_billingFundID', 32):

    SubscriptionUtils.doesUserHavePaymentCardAdded() returns true
    TrialStartedBillingBanner sets subtitle = ''
    The BillingBanner component still renders an empty Text component

    <View style={[styles.flex1, styles.justifyContentCenter]}>
    {typeof title === 'string' ? <Text style={[styles.textStrong, titleStyle]}>{title}</Text> : title}
    {typeof subtitle === 'string' ? <Text style={subtitleStyle}>{subtitle}</Text> : subtitle}
    </View>

    What changes do you think we should make in order to solve the problem?

    We should add a conditional check !!subtitle && before rendering the subtitle component, so when subtitle is an empty string or falsy, it's not rendered at all. This allows the single title element to be properly centered by justifyContentCenter.

    {!!subtitle && (typeof subtitle === 'string' ? <Text style={subtitleStyle}>{subtitle}</Text> : subtitle)}

    <View style={[styles.flex1, styles.justifyContentCenter]}>
    {typeof title === 'string' ? <Text style={[styles.textStrong, titleStyle]}>{title}</Text> : title}
    {typeof subtitle === 'string' ? <Text style={subtitleStyle}>{subtitle}</Text> : subtitle}
    </View>

    What alternative solutions did you explore? (Optional)

    Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

  8. AndrewGable commented on Nov 10, 2025

    @AndrewGable
    Contributor

    Waiting to hear back from PR authors on the linked issue: #74531

    cc @allgandalf @shubham1206agra @danieldoglas

  9. 30 remaining items

  10. eVoloshchak commented on Dec 21, 2025

    @eVoloshchak
    Contributor

    @AndrewGable, this is ready for payment

  11. samranahm commented on Jan 1, 2026

    @samranahm
    Contributor

    @AndrewGable Gentle bump ^

  12. AndrewGable commented on Jan 5, 2026

    @AndrewGable
    Contributor

    @VictoriaExpensify will handle payment, thanks!

  13. samranahm commented on Jan 16, 2026

    @samranahm
    Contributor
  14. samranahm commented on Jan 29, 2026

    @samranahm
    Contributor

    @VictoriaExpensify Could you see this comment?

  15. eVoloshchak commented on Feb 25, 2026

    @eVoloshchak
    Contributor

    @VictoriaExpensify, gentle bump on the above, this is ready for payment

  16. samranahm commented on Mar 10, 2026

    @samranahm
    Contributor

    @AndrewGable @VictoriaExpensify friendly bump, this is ready for payment.

  17. eVoloshchak commented on Mar 25, 2026

    @eVoloshchak
    Contributor

    @mallenexpensify, could you handle the payment on this one, please?

  18. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed
    ReviewingHas a PR in review
    on Mar 26, 2026
  19. melvin-bot commented on Mar 26, 2026

    @melvin-bot

    Triggered auto assignment to @mallenexpensify (Awaiting Payment)

  20. mallenexpensify commented on Mar 26, 2026

    @mallenexpensify
    Contributor

    Payment Summary

    Contributor: @samranahm paid $250 via Upwork
    Contributor+: @eVoloshchak due $250 via NewDot

    ^ Test case created

    Thx! Apologies for the payment delay

  21. trjExpensify commented on Apr 2, 2026

    @trjExpensify
    Contributor

    approved $250 for @eVoloshchak

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 productionBugSomething is broken. Auto assigns a BugZero manager.EngineeringExternalAdded to denote the issue can be worked on by a contributorMonthlyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions