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

Limit locks to a set of defined values #189

Open
jakelangelier opened this issue Mar 15, 2024 · 1 comment
Open

Limit locks to a set of defined values #189

jakelangelier opened this issue Mar 15, 2024 · 1 comment

Comments

@jakelangelier
Copy link

I couldn't find if something similar is already implemented. Apologies if that is the case.

We use the bot to lock workstations. Is there a way to define a list of available devices (in this exemple workstations)
Something like
/lock add wks1
/lock remove wks1
/lock list (would show status of each one too)

Right now i can lock something that is already locked because i made a typo.

@jakelangelier jakelangelier changed the title Limit available locks to a set of defined values Limit locks to a set of defined values Mar 15, 2024
@connorads
Copy link
Owner

Hey @jakelangelier

Thanks for your message. Seems like you're not the only one.

I'm not actively working on Lockbot but you're more than welcome to open a PR to try to add this feature.

I recommend trying to extend the core logic.

There's a test suite for it and you could start by writing some tests for new functionality.

If you get stuck I can have a gander.

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

No branches or pull requests

2 participants