Skip to content
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

Loki: Fix autocomplete situations with multiple escaped quotes #65520

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

svennergr
Copy link
Contributor

@svennergr svennergr commented Mar 29, 2023

What is this feature?

This PR fixes situations where autocomplete was broken for situations with multiple escaped quotes.

Special notes for your reviewer:

  1. Use Grafana with Loki devevn
  2. Start typing the {job="\"grafana/data\"", query.
  3. Without the fix no more suggestions are provided.

@svennergr svennergr changed the title Loki: Fix autocomplete situations with multiple encoded quotes Loki: Fix autocomplete situations with multiple escaped quotes Mar 29, 2023
@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/pr-codeql-analysis-javascript.yml:analyze. As part of the setup process, we have scanned this repository and found 27 existing alerts. Please check the repository Security tab to see all alerts.

@github-actions
Copy link
Contributor

Backend code coverage report for PR #65520
No changes

@github-actions
Copy link
Contributor

Frontend code coverage report for PR #65520

Plugin Main PR Difference
loki 83.72% 83.72% 0%

@svennergr svennergr merged commit b01194f into main Mar 29, 2023
@svennergr svennergr deleted the svennergr/fix-autocompletion branch March 29, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants