Skip to content

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport 85e0767 from #8685.

* memoization plus not setting the user query language

Signed-off-by: Kawika Avilla <[email protected]>

* clean up commented code

Signed-off-by: Kawika Avilla <[email protected]>

* Changeset file for PR #8685 created/updated

* Changeset file for PR #8685 deleted

---------

Signed-off-by: Kawika Avilla <[email protected]>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 85e0767)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

❌ Invalid Changelog Heading

The '## Changelog' heading in your PR description is either missing or malformed. Please make sure that your PR description includes a '## Changelog' heading with proper spelling, capitalization, spacing, and Markdown syntax.

@codecov
Copy link

codecov bot commented Oct 23, 2024

Codecov Report

Attention: Patch coverage is 55.55556% with 12 lines in your changes missing coverage. Please review.

Project coverage is 60.92%. Comparing base (a0b42c6) to head (4dda286).
Report is 3 commits behind head on 2.x.

Files with missing lines Patch % Lines
.../data/public/ui/query_editor/language_selector.tsx 55.55% 7 Missing and 5 partials ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##              2.x    #8688   +/-   ##
=======================================
  Coverage   60.92%   60.92%           
=======================================
  Files        3787     3787           
  Lines       90310    90319    +9     
  Branches    14156    14157    +1     
=======================================
+ Hits        55020    55026    +6     
- Misses      31818    31820    +2     
- Partials     3472     3473    +1     
Flag Coverage Δ
Linux_1 29.03% <ø> (ø)
Linux_2 56.52% <ø> (ø)
Linux_3 37.71% <55.55%> (+<0.01%) ⬆️
Linux_4 29.93% <ø> (ø)
Windows_1 29.04% <ø> (ø)
Windows_2 56.48% <ø> (ø)
Windows_3 37.71% <55.55%> (+<0.01%) ⬆️
Windows_4 29.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@virajsanghvi virajsanghvi merged commit 3cbb65d into 2.x Oct 23, 2024
70 of 73 checks passed
@github-actions github-actions bot deleted the backport/backport-8685-to-2.x branch October 23, 2024 17:15
ruchidh pushed a commit to ruchidh/OpenSearch-Dashboards that referenced this pull request Jul 18, 2025
…) (opensearch-project#8688)

* memoization plus not setting the user query language

* clean up commented code

* Changeset file for PR opensearch-project#8685 created/updated

* Changeset file for PR opensearch-project#8685 deleted

---------

(cherry picked from commit 85e0767)

Signed-off-by: Kawika Avilla <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Signed-off-by: Ruchi Sharma <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autocut Skip the changelog verification check on backports discover for discover reinvent discover-next distinguished-contributor failed changeset v2.18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants