-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD for payment 2023-09-07] [$1000] PDF preview shows blank white area #23327
Comments
Triggered auto assignment to @JmillsExpensify ( |
Bug0 Triage Checklist (Main S/O)
|
@JmillsExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Jumping in to test this now. |
I'm able to reproduce this one, so I'll triage it. Note: The white space is only visible in the preview, and I wasn't able to reproduce the same issue on native iOS. |
Job added to Upwork: https://www.upwork.com/jobs/~01371d83289588168c |
Current assignee @JmillsExpensify is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
I don't know what you are using for rendering, but the sample file has a page of 414 x 12454 points. |
📣 @alexcherut! 📣
|
Contributor details |
|
@JmillsExpensify, @eVoloshchak Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@alexcherut, but it's displayed correctly in preview and in other apps, there definately is a way to render this properly |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (First week) |
Let's close if not reproducible again. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.59-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-09-07. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
For reference, here are some details about the assignees on this issue:
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
|
Regression Test Proposal
Do we agree 👍 or 👎 |
bump ^^ @eVoloshchak @JmillsExpensify |
@JmillsExpensify Can you make the paiement thanks |
@JmillsExpensify Can you please check if possible |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
Agree with the regression test. I'll get that added. In the meantime, here's the payment summary:
@ShogunFire I see you have a thumbs down on the automated message that the urgency bonus doesn't apply. If you disagree, please explain why. |
Upwork job is here: https://www.upwork.com/jobs/~01371d83289588168c. Please apply if not automatically assigned. |
@JmillsExpensify Ahah sorry I don't disagree, I was just not happy about it |
@JmillsExpensify |
Ok thanks all! @DinalJivani Just sent an offer. @ShogunFire Just issued payment. @eVoloshchak I believe you'll create a payment request separately. |
Everyone paid in Upwork is processed. Keeping open for the regression test and @eVoloshchak's payment. |
Not overdue! |
@eVoloshchak Don't forget to finish the regression so we can finish the payment and close this |
@JmillsExpensify All ready for you to finish any payments left and close it! |
Regression test is created and confirmed that there is a NewDot payment. Closing. |
$1,000 payment approved for @eVoloshchak based on BZ summary. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
PDF preview should be shown properly
Actual Result:
PDF preview shows blank white area
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.42-6
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
RPReplay_Final1689772946.mp4
GITK0823.1.MP4
International_Money_Transfer__Send_Money_Abroad_with_Wise__ex-TransferWise_.pdf
Expensify/Expensify Issue URL:
Issue reported by: @DinalJivani
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689772959912119
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: