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

feat(store): add amd rx 6000 series model #832

Merged
merged 6 commits into from Nov 19, 2020

Conversation

rcbevans
Copy link
Contributor

Description

AMD RX 6800/6800XT/6900XT to Series type. Extend .env-example with new MAX_PRICE config. Add AMD reference and Strix LC as model types. Add Sapphire as new brand.

Testing

Verified program runs without error. Tested that filtering series to new AMD series runs, albeit there are no product pages to check.

@rcbevans rcbevans requested a review from jef as a code owner November 18, 2020 10:13
@rcbevans rcbevans changed the title Add initial AMD RX 6000 series model support feat: Add initial AMD RX 6000 series model support Nov 18, 2020
@frotofraggins
Copy link

was this added?

@kckaiwei
Copy link

No, and the models have yet to be updated.

@foxlet
Copy link
Contributor

foxlet commented Nov 18, 2020

The URL for the regular RX 6800 (non-XT) is https://www.amd.com/en/direct-buy/5458373400/us by the way.

@jef
Copy link
Owner

jef commented Nov 18, 2020

Looks great! Thanks for also sorting lexicographically!

jef
jef previously approved these changes Nov 18, 2020
@jef
Copy link
Owner

jef commented Nov 18, 2020

Sorry about the linting. I need to make an update to the max columns. I don't like it being this short.

@tech-degenerate
Copy link

When will this be a feature we can get in a normal update? sorry sort of a noob...

@jef jef changed the title feat: Add initial AMD RX 6000 series model support feat(store): add amd rx 6000 series model Nov 19, 2020
@jef jef merged commit 89a762e into jef:main Nov 19, 2020
@jeff4uraree
Copy link

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

7 participants