-
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-07-12] [HOLD for payment 2023-07-10] [$1000] Date is not being displayed correctly when single digit day is entered #21404
Comments
Triggered auto assignment to @kevinksullivan ( |
Bug0 Triage Checklist (Main S/O)
|
@kevinksullivan It gets truncated when you select a date between 1 to 9. |
@kevinksullivan Whoops! This issue is 2 days overdue. Let's get this updated quick! |
ha, got it. I updated the steps to make that a bit clearer. |
Reproduced and sending to the pool |
Job added to Upwork: https://www.upwork.com/jobs/~01046c3f7c49b615ed |
Current assignee @kevinksullivan is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.The date is not fully visible What is the root cause of that problem?It's this line where we're using App/src/components/MenuItem.js Line 83 in 48d4187
Text is smaller than the actual width, the Text will show ellipsis.
What changes do you think we should make in order to solve the problem?We just need to make the App/src/components/MenuItem.js Line 85 in 48d4187
What alternative solutions did you explore? (Optional)We might want to check other places, it might have the same issues and can apply the same fix. |
@dukenv0307's proposal here looks good. I think we should focus on 🎀 👀 🎀 C+ reviewed |
Triggered auto assignment to @cristipaval, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Date is not being displayed correctly when single digit day is entered What is the root cause of that problem?The actual problem with our We do not set When set full width or flex @mananjadhav What changes do you think we should make in order to solve the problem?When we set We can remove this style or replace a property - styles.pre or - styles.pre
+ styles.noWrap It always works fine in a single line because we already use What alternative solutions did you explore? (Optional)
|
📣 @dukenv0307 You have been assigned to this job by @cristipaval! |
Hi @dukenv0307 , what melvin said ^ 😄 . Please apply for the job in upwork as I'm not sure what your name would be for searching it. Thanks! |
Offer sent for C+ @mananjadhav |
@mananjadhav, @cristipaval, @kevinksullivan, @dukenv0307 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@kevinksullivan did you get a chance to look at the previous comment? |
@mananjadhav so are you asking for the standard $1,000 then? |
@kevinksullivan Yes the standard 1000$. |
@kevinsullivan did you get a chance to look at my previous comment? This is pending for payout. |
Please let me know what I need to do PS I think you meant @kevinksullivan. |
📣 @kevinsullivan! 📣
|
Apologies for the issue. Yes I meant to tag @kevinksullivan |
Thank you, no problem
…On Tue, Jul 18, 2023, 12:44 PM Manan ***@***.***> wrote:
Apologies for the issue. Yes I meant to tag @kevinksullivan
<https://github.com/kevinksullivan>
—
Reply to this email directly, view it on GitHub
<#21404 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAVEX5MN34P5ZEUHUZFIYBDXQ24QVANCNFSM6AAAAAAZRTBQ7I>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Paid out |
Thanks the payout @kevinksullivan. Is it possible to do a refund without additional charges? I am currently raising requests from NewDot. |
So you already got paid for this in newdot @mananjadhav ? |
DMed you @kevinksullivan. |
just sent a refund request @mananjadhav |
Thanks @kevinksullivan. |
Approved 1000 to Manan based on #21404 (comment), which has been refunded on Upwork! |
@kevinksullivan The PR is merged within 3 working days so I think I am eligible for a bonus timeline |
@dukenv0307 We missed a case here, hence this isn't eligible for the timeline bonus. |
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:
The date is fully visible
Actual Result:
The date is not fully visible
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.31-2
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
WhatsApp.Video.2023-06-17.at.15.20.12.mp4
SVDK7066.1.MP4
Expensify/Expensify Issue URL:
Issue reported by: @aman-atg
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686991387615589
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: