-
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
[HOLD for payment 2024-06-28] [$250] [Search v1] Live search updates #41626
Comments
@luacmartins Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@luacmartins Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Taking off hold, we'll start implementation shortly. |
@luacmartins |
No API needed for this one. We only need to update Onyx so that any incoming update is processed in a way that it also updates the data in the |
@Kicu I assigned you to this issue. Let me know if you can't work on it. |
Triggered auto assignment to @Christinadobrzyn ( |
Will work on it 👍 |
@luacmartins do you mind checking my comment here? https://docs.google.com/document/d/1ldZ85fyQO4662hLZRNFfm0_IxaP7-PWvaqc8Z33jsLQ/edit?disco=AAABJtWc-70 |
@Kicu, @luacmartins, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick! |
looks like @luacmartins and @Kicu are working through the comments in the design doc |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
This comment was marked as off-topic.
This comment was marked as off-topic.
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.85-7 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-06-28. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Payment Summary
BugZero Checklist (@Christinadobrzyn)
|
Hum... the upwork contract has an offer sent to @rayane-djouah Are we paying out anyone other than @allroundexperts and @jnowakow for this? |
@Christinadobrzyn I'm from SoftwareMansion, an expert agency, so I'm not receiving direct payments 😅 |
Thanks @jnowakow! This explains why we sent an offer to @rayane-djouah so I'm going to withdraw that offer. Payment summary here - #41626 (comment) @allroundexperts can you let me know about a regression test? |
moving this to weekly since all we're waiting on is a regression test, if applicable. |
No regression tests needed. We'll add them as part of the project wrap up. We can close this then. Thanks everyone! |
$250 approved for @allroundexperts |
Section from design doc - https://docs.google.com/document/d/1ldZ85fyQO4662hLZRNFfm0_IxaP7-PWvaqc8Z33jsLQ/edit#bookmark=kix.79jwwhj0fjyz
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @allroundexpertsThe text was updated successfully, but these errors were encountered: