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

Remove Filter input as this component is now part of our internal design system #692

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

maximedegreve
Copy link
Contributor

@maximedegreve maximedegreve commented Dec 12, 2023

Copy link
Member

@lesliecdubs lesliecdubs left a comment

Choose a reason for hiding this comment

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

I'm not an expert on how content from this repo appears in primer.style so you may want an additional approval, but this generally LGTM

@@ -544,7 +544,7 @@ If one of the actions is heavily used, pull it out for easier access. Do not pul

### Filtering

A [filter input](/components/filter-input) is used to write a query and only show rows that
A [filter input (internal)](https://ui.githubapp.com/storybook/?path=/docs/recipes-filter--readme) is used to write a query and only show rows that
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
A [filter input (internal)](https://ui.githubapp.com/storybook/?path=/docs/recipes-filter--readme) is used to write a query and only show rows that
A [filter input (GitHub staff only)](https://ui.githubapp.com/storybook/?path=/docs/recipes-filter--readme) is used to write a query and only show rows that

I believe this is how we typically denote content in primer.style that is inaccessible to non-Hubbers.

@maximedegreve maximedegreve merged commit 855e7c6 into main Dec 12, 2023
4 checks passed
@maximedegreve maximedegreve deleted the feature/remove-filter-input branch December 12, 2023 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants