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

Add GX6 communication dongle support #2

Closed
wants to merge 22 commits into from

Conversation

JovannMC
Copy link
Collaborator

@JovannMC JovannMC commented Mar 1, 2024

This PR adds support for the GX6 communication dongle, modifying my proof-of-concept script made in Python and converting it to JavaScript while adding features required for the program to support it.

PR also includes some QOL and maintenance changes.

Electron wasn't installed as a dev dependency, leading to errors when trying to run it and would require installing electron globally (not ideal).

Also cleaned up package.json with leftover code from BracketBrowser
Getting ready to add support for GX6 dongle
Slightly restructured script, can now access some data outside the dongle.js
Still need to create more methods to export (eg setting individual tracker settings)
idk why this kinda triggered me lol
gotta follow camelCaseConventionsFr

though some users who set it might have their smoothinput setting reset
me when inconsistent
forgot about index.js
Final commit for using camelCase for consistency :p

i hope this doesn't screw me up with merge conflicts later on.
this took me forever fixing bugs. please help. i've been at this for hours.
i hate merge conflicts. i feel like everything broke faksd
@JovannMC
Copy link
Collaborator Author

JovannMC commented Mar 7, 2024

Been working on a NPM package with improved code - will implement support with new PR when it's done soon

@JovannMC JovannMC closed this Mar 7, 2024
@JovannMC JovannMC deleted the gx6-support branch March 29, 2024 00:16
JovannMC added a commit that referenced this pull request May 3, 2024
Add JSON for Japanese locale
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.

1 participant