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

fix: update gigabyte model vision oc #550

Merged
merged 2 commits into from Oct 20, 2020
Merged

Conversation

KazanK
Copy link
Contributor

@KazanK KazanK commented Oct 20, 2020

Description

Purely an informational change for anyone looking to filter by Gigabyte's Vision OC card. Following the readme's filter guide for this model results in a filter for a card that doesn't exist.

Testing

Running this while filtering by gigabyte brand results in searches for vision oc cards (along with other Gigabyte cards, as expected), but filtering by the vision model like the readme says doesn't result in any searches. Filtering by the vision oc model works as expected.

@KazanK KazanK requested a review from jef as a code owner October 20, 2020 06:57
@jef
Copy link
Owner

jef commented Oct 20, 2020

Ah, makes sense. Thanks for updating!

@jef jef changed the title fix: The Gigabyte vision model should be "vision oc" when filtering by model fix: update gigabyte model vision oc Oct 20, 2020
@jef jef merged commit 4fefe22 into jef:main Oct 20, 2020
@KazanK KazanK deleted the gigabyte-vision-oc branch October 21, 2020 02:58
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.

None yet

2 participants