Skip to content

Added new Rival 3 PID#155

Merged
DarthAffe merged 1 commit intoDarthAffe:Developmentfrom
RafeeSamith:Development
Oct 25, 2020
Merged

Added new Rival 3 PID#155
DarthAffe merged 1 commit intoDarthAffe:Developmentfrom
RafeeSamith:Development

Conversation

@RafeeSamith
Copy link
Contributor

The Rival 3 PID changed from 0x1824 to 0x184C after a firmware update. I've updated DeviceChecker to support it and also left the old one in there

The Rival 3 PID changed from 0x1824 to 0x184C after a firmware update
("Rival 500", RGBDeviceType.Mouse, 0x170E, SteelSeriesDeviceType.TwoZone, "default", @"Mice\Rival500", MOUSE_TWO_ZONE),
("Rival 310", RGBDeviceType.Mouse, 0x1720, SteelSeriesDeviceType.TwoZone, "default", @"Mice\Rival310", MOUSE_TWO_ZONE),
("Rival 3", RGBDeviceType.Mouse, 0x1824, SteelSeriesDeviceType.ThreeZone, "default", @"Mice\Rival3", MOUSE_THREE_ZONE),
("Rival 3 (Old Firmware)", RGBDeviceType.Mouse, 0x1824, SteelSeriesDeviceType.ThreeZone, "default", @"Mice\Rival3", MOUSE_THREE_ZONE),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Old PID

("Rival 310", RGBDeviceType.Mouse, 0x1720, SteelSeriesDeviceType.TwoZone, "default", @"Mice\Rival310", MOUSE_TWO_ZONE),
("Rival 3", RGBDeviceType.Mouse, 0x1824, SteelSeriesDeviceType.ThreeZone, "default", @"Mice\Rival3", MOUSE_THREE_ZONE),
("Rival 3 (Old Firmware)", RGBDeviceType.Mouse, 0x1824, SteelSeriesDeviceType.ThreeZone, "default", @"Mice\Rival3", MOUSE_THREE_ZONE),
("Rival 3", RGBDeviceType.Mouse, 0x184C, SteelSeriesDeviceType.ThreeZone, "default", @"Mice\Rival3", MOUSE_THREE_ZONE),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

New PID

@DarthAffe
Copy link
Owner

Oo didn't expect that ever to happen. Thanks for the contribution!

@DarthAffe DarthAffe merged commit bfb024d into DarthAffe:Development Oct 25, 2020
@RafeeSamith
Copy link
Contributor Author

no problem c:

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