Features: Easier input of schedule & Allows blacklist to be loaded from text file#44
Open
shivgodhia wants to merge 4 commits intoandreasgrill:masterfrom
Open
Features: Easier input of schedule & Allows blacklist to be loaded from text file#44shivgodhia wants to merge 4 commits intoandreasgrill:masterfrom
shivgodhia wants to merge 4 commits intoandreasgrill:masterfrom
Conversation
Owner
|
Thanks for your commits, I like the idea of simplifying the configuration! |
Author
You're welcome! I have resolved the changes and it should work now, though I must admit I don't use Self Control anymore so I didn't manage to test it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi, I made a simple but useful change to the code.
Schedule Input
Now, in
config.jsonfor scheduling, instead of just writing a number for the day, such as1or3for Monday and Wednesday respectively, you have more options:"weekdays"will use Monday to Friday"weekends"will use Saturday and Sunday[1,3,5]for Monday, Wednesday and Friday or[1,2,3,4,5,6], etc however you wish.It still supports the earlier way of using the integer representation of the day, as well as letting the value be
None(which will be all 7 days of the week).Blacklist import
Now you can also import a blacklist from a text file or a few text files so it is easier to fill in the
config.jsonand even create separate blacklist categories to be toggled on and off.