-
Notifications
You must be signed in to change notification settings - Fork 3.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
fix: Query missing in AUTO_COMPLETE_SHOW event #24898
Conversation
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/5398435600. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/5398435600.
To know the list of identified flaky tests - Refer here |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/5398435600. |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/5399817644. |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/5399904862. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/5399904862. |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/5462521859. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/5462521859. |
/build-deploy-preview skip-tests=true env=release |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/5463459538. |
Deploy-Preview-URL: https://appsmith-69aj8hflo-get-appsmith.vercel.app |
/ok-to-test |
/build-deploy-preview skip-tests=true env=release |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/5464375093. |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/5464376407. |
Deploy-Preview-URL: https://appsmith-r7yzd6ni1-get-appsmith.vercel.app |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/5464375093. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Query missing in AUTO_COMPLETE_SHOW event
PR fixes following issue(s)
Fixes #23838
Media
Type of change
Testing
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity:
Test Plan Approved
label after Cypress tests were reviewedTest Plan Approved
label after JUnit tests were reviewed