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

Filter API Docs #9202

Merged
merged 4 commits into from
Dec 9, 2020
Merged

Filter API Docs #9202

merged 4 commits into from
Dec 9, 2020

Conversation

kaitlincart
Copy link
Contributor

From user feedback, I tried to update the filter api docs to be a more clear on how to create expressions.

@kaitlincart kaitlincart requested review from mkeeler and a team November 16, 2020 18:29
@github-actions github-actions bot added the type/docs Documentation needs to be created/updated/clarified label Nov 16, 2020
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
Co-authored-by: Blake Covarrubias <blake@covarrubi.as>
Copy link
Contributor

@freddygv freddygv left a comment

Choose a reason for hiding this comment

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

Thanks for going through these! It was definitely challenging to get started with filtering.

Left some minor suggestions but in general LGTM.

website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
website/pages/api-docs/features/filtering.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@freddygv freddygv left a comment

Choose a reason for hiding this comment

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

Thanks for going through these! It was definitely challenging to get started with filtering.

Left some minor suggestions but in general LGTM.

Copy link
Contributor

@freddygv freddygv left a comment

Choose a reason for hiding this comment

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

Thanks for going through these! It was definitely challenging to get started with filtering.

Left some minor suggestions but in general LGTM.

Co-authored-by: Freddy <freddygv@users.noreply.github.com>
@hashicorp-ci
Copy link
Contributor

🍒 If backport labels were added before merging, cherry-picking will start automatically.

To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/295344.

@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit 7ea383f onto stable-website succeeded!

hashicorp-ci pushed a commit that referenced this pull request Dec 9, 2020
* reorganize for clarity and update for value syntax

* fix quotes around value

* Apply suggestions from code review

Co-authored-by: Blake Covarrubias <blake@covarrubi.as>

* Apply suggestions from code review

Co-authored-by: Freddy <freddygv@users.noreply.github.com>

Co-authored-by: Blake Covarrubias <blake@covarrubi.as>
Co-authored-by: Freddy <freddygv@users.noreply.github.com>
@hashicorp-ci
Copy link
Contributor

🍒✅ Cherry pick of commit 7ea383f onto release/1.9.x succeeded!

hashicorp-ci pushed a commit that referenced this pull request Dec 9, 2020
* reorganize for clarity and update for value syntax

* fix quotes around value

* Apply suggestions from code review

Co-authored-by: Blake Covarrubias <blake@covarrubi.as>

* Apply suggestions from code review

Co-authored-by: Freddy <freddygv@users.noreply.github.com>

Co-authored-by: Blake Covarrubias <blake@covarrubi.as>
Co-authored-by: Freddy <freddygv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants