-
Notifications
You must be signed in to change notification settings - Fork 3k
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 2024-02-26] Simplify native attachment gallery #36034
Comments
@tgolen I can't submit the labels
|
Job added to Upwork: https://www.upwork.com/jobs/~01d28994a18233cccb |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @rushatgabhane ( |
Triggered auto assignment to @kadiealexander ( |
@hannojg I added the labels! |
Big thanks! |
@kirillzyusko Can you please comment here so the issue can get assigned to you? Thanks! |
@hannojg yes, sure I can work on this ticket |
@pecanoro Could you assign me to the issue? Thank you! |
I've added you 👍 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.42-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 2024-02-26. 🎊 For reference, here are some details about the assignees on this 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:
|
Payment Summary
BugZero Checklist (@kadiealexander)
|
No offending PR. This is an improvement for the attachment gallery components.
No need new regression step. We already have the test rail for attachment under Conversation > Attachments and Downloads. |
@kadiealexander Could you give the payment summary for the manual request in NewDot? Thank you! |
Payouts due:
|
$500 approved for @mollfpr based on summary above. |
Internal tracking issue for cleaning up the native attachment gallery/carousel.
In a recent succession of PRs a lot of work has happened on the native attachment carousel and we need to cleanup the code a bit.
Recent PRs were:
MultiGestureCanvas
to hooks and improve code #33756Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @kadiealexanderThe text was updated successfully, but these errors were encountered: