Skip to content

[Due for payment 2026-10-14] [Insights Polish] Bar chart's bars #102863

Closed

Description

@JmillsExpensify
Issue OwnerCurrent Issue Owner: @mallenexpensify

Activity

  1. added
    ImprovementItem broken or needs improvement.
    InternalRequires API changes or must be handled by Expensify staff
    on Oct 2, 2026
  2. mhawryluk commented on Oct 5, 2026

    @mhawryluk
    Member

    I'll take this one

  3. mhawryluk commented on Oct 5, 2026

    @mhawryluk
    Member
  4. MelvinBot commented on Oct 5, 2026

    @MelvinBot
    Contributor

    Assigned truph01 to this issue.


    view run

  5. 11 remaining items

  6. melvin-bot commented on Oct 13, 2026

    @melvin-bot

    Triggered auto assignment to @mallenexpensify (Awaiting Payment)

  7. melvin-bot commented on Oct 13, 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
  8. truph01 commented on Oct 14, 2026

    @truph01
    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.

      Link to the comment on the PR:
      Link to the comment on the Issue:

    • [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:

    • [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. Enable the insightsPage beta.
    2. Open Insights on wide and narrow layouts.
    3. Verify Top spenders and Top merchants are vertical bars, with a different color per bar.
    4. Verify there are no labels under the bars.
    5. Verify bars fill the plot with 16px gaps and 4px corners.
    6. Hover and click a bar. Verify the hit box covers full y axis, not just the bar. Verify the tooltip shows, and clicking opens the filtered Spend page.
    7. Open Spend, group expenses by category, and switch to the Bar view.
    8. Verify labels show under the bars, rotating or shortening as before.
    9. Verify that with many groups, the bars stay separated and don't overlap.
    10. Open a Spend bar chart with a single group. Verify the bar is centered, has max-width set and nothing is cut off.

    Do we agree 👍 or 👎

  9. truph01 commented on Oct 14, 2026

    @truph01
    Contributor

    @mallenexpensify This is part of project #317. As we can see, a few completed issues were paid at the old rate of $250.

  10. melvin-bot commented on Oct 16, 2026

    @melvin-bot

    @mallenexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

  11. mallenexpensify commented on Oct 16, 2026

    @mallenexpensify
    Contributor

    Payment Summary

    Contributor+: @truph01 due $250 via NewDot

  12. garrettmknight commented on Oct 19, 2026

    @garrettmknight
    Contributor

    $250 approved for @truph01

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 productionDailyKSv2ImprovementItem broken or needs improvement.InternalRequires API changes or must be handled by Expensify staff

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions