-
Notifications
You must be signed in to change notification settings - Fork 1
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
Rework #1
Rework #1
Conversation
…m recipes, name consistency changes
@Misfiy review my pr !! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay. Hope that project is compiling and everything is working as expected
Did you test all changes at all? |
everything before your first review was tested, everything after wasn't, if you want I can quickly check if everything is still fine |
Also it will be nice if make all features disabled by default, for example, auto nuke, periodical broadcast message and etc |
sounds good, I will get on it when I can |
actually, I'll leave some features enabled like the disarmed escapes since disabling it would require me to delete that part from the config and it serves as a guide for people on how to input the values correctly |
No description provided.