-
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-26] [$1000] The chat composer is hidden and is not scrollable on a new page with a description/date #19389
Comments
Triggered auto assignment to @NicMendonca ( |
This comment was marked as outdated.
This comment was marked as outdated.
Going OOO until June 5th so assigning a buddy to this GH to watch over the issue. I'll pick it back up when I am back from OOO/ if this is still open. |
Triggered auto assignment to @maddylewis ( |
Bug0 Triage Checklist (Main S/O)
|
Job added to Upwork: https://www.upwork.com/jobs/~01beba9fa9a0c7d961 |
Current assignee @maddylewis is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif ( |
Triggered auto assignment to @madmax330 ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.The chat composer is hidden and not scrollable on the money request page. What is the root cause of that problem?The From the code listing at https://github.com/Expensify/App/blob/main/src/components/MoneyRequestHeader.js#L177-L193, we can see that each piece of information is laid out one after the other which occupies too much vertical space for smaller screens. What changes do you think we should make in order to solve the problem?Automatically adjust the layout of the For the compact view, we could one or more of the following:
What alternative solutions did you explore? (Optional)Alternatively, always display a compact version of the money request header for mobile screens. money-request-header.mp4 |
i'll let @aimane-chnaif review this most recent proposal and confirm if that's how we should move forward. |
Waiting on proposal reviews @aimane-chnaif |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Current assignee @NicMendonca is eligible for the External assigner, not assigning anyone new. |
Current assignee @aimane-chnaif is eligible for the External assigner, not assigning anyone new. |
^ sorry, was just checking to see if that would re-create the job in Upwork since the original job post expired 👼 |
@ShogunFire here's the job post! https://www.upwork.com/jobs/~016e8237e6e708015c @aimane-chnaif are you requesting payment via Expensify? |
@NicMendonca Sorry but can you invite me ? I have already applied on another issue so I think they have stored my profile for next time and I don't want to use 16 connects on all my current issues |
Still upwork |
@ShogunFire I can't find you to invite, you are not searchable via GH username. Can you please link your Upwork or apply to the job here: https://www.upwork.com/jobs/~016e8237e6e708015c @aimane-chnaif thanks! Just sent you the offer |
@aimane-chnaif you've been paid 🎉 @ShogunFire just need you to accept the offer, please |
@aimane-chnaif bump on the BZ checklist also: #19389 (comment) |
@ShogunFire you've been paid! 🎉 |
@aimane-chnaif bump on BZ checklist pls! |
This is a new feature, not bug. |
everyone has been paid! |
@NicMendonca I'm the external reporter. I think I haven't been paid for this yet. |
@priya-zha sorry! I was getting confused by another GH. Can you please apply to the job? https://www.upwork.com/jobs/~016e8237e6e708015c |
@NicMendonca submitted the proposal. Thanks |
@priya-zha you've been paid! sorry again! |
@NicMendonca Hello, sorry for the late request but could you put the upwork mission as completed ? Because it makes my upwork profile worse otherwise. Thanks |
I took care of this! |
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:
I'll settle up elsewhere
Expected Result:
The chat composer should not be hidden on a new page with description and date after sending money
Actual Result:
The chat composer is hidden on a new page with description and date after sending money
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.16-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
composer.mp4
az_recorder_20230522_112100.1.mp4
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684475099942009
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: