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

JS: Move experimental notice to the bottom of the ML-powered query help #7744

Merged
merged 1 commit into from Jan 25, 2022

Conversation

@henrymercer
Copy link
Contributor

@henrymercer henrymercer commented Jan 25, 2022

The Code Scanning UI shows just the first paragraph of the query help as a summary, until a user chooses to expand the help. We decided it was more useful to display the standard query help in this summary compared to the experimental query notice, since there is already a notice about experimental queries on the alert show page.

The Code Scanning UI shows just the first paragraph of the query help
as a summary, until a user chooses to expand the help.
We decided it was more useful to display the standard query help in this
summary compared to the experimental query notice, since there is
already a notice about experimental queries on the alert show page.
Copy link
Contributor

@AlonaHlobina AlonaHlobina left a comment

Looks great! Thank you!

@henrymercer henrymercer merged commit 15aa09f into main Jan 25, 2022
16 checks passed
@henrymercer henrymercer deleted the henrymercer/js-atm-tweak-query-help branch Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants