-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[$250] [HOLD for payment 2024-07-22] Expense - Blank area below report header in the expense report #44434
Comments
Triggered auto assignment to @alexpensify ( |
Triggered auto assignment to @MariaHCD ( |
👋 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:
|
@alexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors. |
We think this issue might be related to the #collect project. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Blank area below report header in the expense report What is the root cause of that problem?We are adding padding of App/src/components/MoneyReportHeader.tsx Line 378 in 57e5d0e
What changes do you think we should make in order to solve the problem?Remove What alternative solutions did you explore? (Optional) |
The issue is introduced by this PR. I can raise a quick pr if the author is not available at the time. |
Investigating. |
@MariaHCD Agree. I'll raise PR in several hours. |
@tienifr any update here? Thanks! |
I believe the payment date has been moved to 7/22, and will prepare accordingly. |
To prepare for the payment process, I believe that this UI update is a dupe of #39989. I asked the BZ team assigned to that GH for feedback. |
Payment Summary
BugZero Checklist (@alexpensify)
|
I'll work on the payment process tomorrow. |
Job added to Upwork: https://www.upwork.com/jobs/~012b0cf135dfea0be9 |
Current assignee @dukenv0307 is eligible for the External assigner, not assigning anyone new. |
Payouts due: 2024-07-22
The Upwork job is here. I noticed @dukenv0307 was part of the review process for this GH. Please accept, and I can proceed with the next steps. Thanks! |
Is this job done I am looking for work to do? |
📣 @AirTechWick! 📣
|
@AirTechWick - Yes, sorry for the confusion. The recent tag updates addressed payment. |
@dukenv0307 - please accept the invite in Upwork. After, I can complete the payment process. Thanks! |
No update yet in Upwork. |
@alexpensify Sorry for the delay, I accepted the UW offer Thank you! |
Thank you! I've completed the payment process in Upwork and updated the payment summary - @tienifr, please submit a request in Chat-- thanks! |
$250 approved for @tienifr |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.2-0
Reproducible in staging?: Y
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
There will be no blank area below the report header.
Actual Result:
There is small blank area below the report header.
This issue only happens in expense report.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6524811_1719378867306.20240626_131015.mp4
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: