Skip to content

Added 2 new Arguments: '--skip-printout' & '--collection'#12

Open
Whatchawnt wants to merge 18 commits intorobobeaver6:masterfrom
Whatchawnt:master
Open

Added 2 new Arguments: '--skip-printout' & '--collection'#12
Whatchawnt wants to merge 18 commits intorobobeaver6:masterfrom
Whatchawnt:master

Conversation

@Whatchawnt
Copy link
Copy Markdown
Contributor

[Script - Changes]
1 - Added the ability to skip the printout of the playlist to the console with the '--skip-printout' option.
2 - Added an option '--collection' which takes an argument of a Collection Name, in order to only grab items from the specified collection by name in order to add them to the playlist generation. The collection name provided will be used to query through each associated library, filtering and grabbing items who are part of a collection with the same name to add them to the playlist.

[ReadMe - Changes]
1 - Added the help details and a usage example for '--skip-printout' argument.
2 - Added the help details and a usage example for '--collection' argument.

1. Fixed a bug that resulted in a NameError when the provided users are invalid.
2. Added Resource argument as a requirement for server connection method (due to the switching of users).
3. Added a check for when all users entered are invalid home users.
4. Minor changes to comments.
1. Added "--resource" argument to requests via server connection method (now a required argument).
2. Added Some examples of using "https" with server connection method.
Minor spelling fix
Minor change, changed line 1281 to remove str conversion.

From: logger.debug(f'\nHome User [matched]: {str(homeUser)}\n')
To: logger.debug(f'\nHome User [matched]: {homeUser}\n')
Cleanup commented out code.
Display "Error - No Valid Home Users Submitted." Error only when the homeusers argument is passed into the script.
Fixed a bug where some movies would be prevented from being added to a playlist due to an already added item being attemptedly added multiple times; however, since it is already in the playlist it was not added again but still caused the playlist count to be lower than expected at times due to this.
Added a check to make sure a user entered either a --adminuser argument or a --homeusers argument.
Added the ability to purge a playlist without needing to provide the --select-library, --allshows, or --allmovies arguments.
1. Added a check to no longer allow the use of the "--purge" argument alongside the following arguments: "--select-library", "--allshows", or "--allmovies"
2. Minor changes to comments.
Forgot additional parenthesis around if statement
updated references of "--admin" to "--adminuser" in examples.
1. Fixed argument references.
2. Updated examples and comments.
Added comma after "--exclude-library"
1 - Added the ability to skip the printout of the playlist to the console with the '--skip-printout' option.
2 - Added an option '--collection' which takes an argument of a Collection Name, in order to only grab items from the specified collection by name in order to add them to the playlist generation. The collection name provided will be used to query through each associated library, filtering and grabbing items who are part of a collection with the same name to add them to the playlist.
1 - Added the help details and a usage example for '--skip-printout' argument.
2 - Added the help details and a usage example for '--collection' argument.
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

Successfully merging this pull request may close these issues.

1 participant