-
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
[PAID][Search v2.1][Web-E] Deprecate old Search params #45029
Comments
Triggered auto assignment to @slafortune ( |
Still on hold |
We're good to review the hold, yeah? |
Not yet, the holding PR is still in draft |
Just in case - adding another BZ - I'll be out until 8/21 and will check in on this then. |
That PR is not related to this issue. I still haven't worked on this one. |
Since this is all internal, I'll unassign you from this issue. |
@luacmartins I reviewed the PR, which has now been in production for a week. However, I wasn't assigned to the issue or involved in the proposal review. Should I be assigned to this issue for compensation, or given the simple nature of the PR, can we close the issue and move on? Thank you. |
Triggered auto assignment to @strepanier03 ( |
Thank you 🙏 |
Regression Test Proposal
Do we agree? 👍 or 👎 |
No need to add the steps above, we're covering them as part of the project wrap up. |
Thanks @luacmartins, was just going to close the GH and saw you took care of that too, appreciate it! |
All right, I've paid and closed the contract and this is ready to be closed as well. Thanks everyone! |
Implement this section of the design doc.
Holding on:
Issue Owner
Current Issue Owner: @strepanier03The text was updated successfully, but these errors were encountered: