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 autorebuild, patrolread, and foreignautoimport to controller #9

Merged
merged 3 commits into from
Apr 12, 2023

Conversation

dgilbert101
Copy link
Contributor

Nice work on the module. I had the need for additional functionality to the controller, so I added the following functionality;

  • autorebuild: current auto rebuild state (also setter)
  • foreignautoimport: imports foreign configuration automatically at boot (also setter)
  • patrolread: current patrol read settings (also setter)
  • set_patrolread: configures patrol read state and schedule
  • patrolread_start: starts a patrol read on the controller
  • patrolread_pause: pauses patrol read on the controller
  • patrolread_resume: resumes patrol read on the controller
  • patrolread_stop: stops patrol read if running on the controller
  • patrolread_running: check if patrol read is running on the controller

@coveralls
Copy link

Coverage Status

Coverage: 55.141% (-0.7%) from 55.874% when pulling 98d0605 on liveaction:develop into f0bdceb on Naudit:develop.

@ralequi ralequi merged commit ece8172 into Naudit:develop Apr 12, 2023
@ralequi
Copy link
Member

ralequi commented Apr 12, 2023

I might change a little bit on the future (adding some python-typing etc) but I think for now is ok

Thanks for your contribution @dgilbert101 !

@ralequi ralequi self-assigned this Apr 12, 2023
@ralequi ralequi added enhancement New feature or request good first issue Good for newcomers labels Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants