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

Query Parameter documentation updates #195

Merged
merged 5 commits into from
Feb 13, 2019
Merged

Query Parameter documentation updates #195

merged 5 commits into from
Feb 13, 2019

Conversation

susodapop
Copy link
Contributor

@susodapop susodapop commented Feb 12, 2019

This includes updates reflecting the new dashboard parameter UX in version 7. Also includes minor tweaks to the introductory paragraph.

@ghost ghost assigned susodapop Feb 12, 2019
@ghost ghost added the in progress label Feb 12, 2019
@susodapop susodapop requested a review from ranbena February 12, 2019 12:49
Copy link
Contributor

@ranbena ranbena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Left some comments 👍

+ **Value Source** is where you choose your preferred mapping. Click the pencil glyph to open the mapper settings.

### Value Source Options

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think value source popover screenshot would be beneficial here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good thinking. I will add this as well.

@ghost ghost assigned arikfr Feb 13, 2019
@arikfr
Copy link
Member

arikfr commented Feb 13, 2019

Because we released this already I didn't want this to wait any longer. I did a few changes (updated text re. default value and callout about version) and going to merge this.

Let's have a follow up with Ran's other comments. Also I think we should have a word on how this used to work (the global checkbox) and how existing dashboards transition into this.

@arikfr arikfr merged commit 03997da into master Feb 13, 2019
@arikfr arikfr deleted the new-parameter-ux branch February 13, 2019 06:58
@ghost ghost removed the in progress label Feb 13, 2019
@arikfr arikfr restored the new-parameter-ux branch February 13, 2019 06:58
@pr-triage pr-triage bot added the PR: merged label Feb 13, 2019
@susodapop susodapop deleted the new-parameter-ux branch February 19, 2019 11:49
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.

3 participants