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

Move static attributes from GlobalConfig to controller params #62

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

laracroft37
Copy link
Contributor

Update key-value pairs in the params dictionary for a generic controller. We can now set click timeout and other settings for each separate controller instance.

Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 85.10%. Comparing base (f175155) to head (0e823d0).
Report is 33 commits behind head on master.

Files with missing lines Patch % Lines
guibot/controller.py 80.00% 4 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (f175155) and HEAD (0e823d0). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (f175155) HEAD (0e823d0)
6 3
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   90.16%   85.10%   -5.06%     
==========================================
  Files          18       17       -1     
  Lines        4492     4598     +106     
==========================================
- Hits         4050     3913     -137     
- Misses        442      685     +243     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pevogam pevogam self-requested a review November 27, 2024 16:39
@pevogam pevogam added the good first issue Good for newcomers label Nov 27, 2024
@laracroft37 laracroft37 force-pushed the controller-settings branch 12 times, most recently from 638f299 to f835d44 Compare November 28, 2024 18:43
guibot/region.py Fixed Show resolved Hide resolved
Copy link
Member

@pevogam pevogam left a comment

Choose a reason for hiding this comment

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

I have left a few comments below.

guibot/config.py Outdated Show resolved Hide resolved
guibot/controller.py Outdated Show resolved Hide resolved
guibot/controller.py Show resolved Hide resolved
guibot/region.py Fixed Show resolved Hide resolved
guibot/region.py Outdated Show resolved Hide resolved
Copy link
Member

@pevogam pevogam left a comment

Choose a reason for hiding this comment

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

Ok, how about a rename for clarity below?

guibot/controller.py Outdated Show resolved Hide resolved
@laracroft37 laracroft37 force-pushed the controller-settings branch 4 times, most recently from c589a95 to ce53b4d Compare November 29, 2024 11:31
Copy link
Member

@pevogam pevogam left a comment

Choose a reason for hiding this comment

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

LGTM, let's just wait for the tests to pass one the mirror servers are working again.

Update key-value pairs in the params dictionary for a generic
controller. We can now set click timeout and other settings
for each separate controller instance.

Localize delay_before_keys to display controllers moving it
away from the region class as it is still relate to key typing
configuration.

rename toggle_delay to mouse_toggle_delay
@pevogam pevogam merged commit 0e0e59c into intra2net:master Dec 6, 2024
8 of 10 checks passed
@laracroft37 laracroft37 deleted the controller-settings branch December 10, 2024 00:56
@laracroft37 laracroft37 restored the controller-settings branch December 10, 2024 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide click_delay and other DC instance specific settings to replace the global config ones
2 participants