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

Merge up 0.6.x to master #209

Merged
merged 209 commits into from
Apr 17, 2021
Merged

Merge up 0.6.x to master #209

merged 209 commits into from
Apr 17, 2021

Conversation

glensc
Copy link
Collaborator

@glensc glensc commented Apr 9, 2021

This creates a merge up so that master can have development done for 0.6.x development.

This PR includes following 0.6.x releases:

glensc added 30 commits March 20, 2021 23:42
This is to avoid dependency loop
Use markdown reflinks for readability
Too specific right now, and not the goal
This makes log entries INFO and above also appear on console
Status messages are printed via logger
As all INFO level messages are printed to console,
this gets rather noisy.
This ensures we don't accidentally include .env, requests_cache, or
config.json to docker image.
@wandabastyle
Copy link

So you asked for testing of the main branch to ultimatily merge the code into master, right?

Here is what I get so far (all using the new metadata agents inside plex for tv & movies):

Syncing of the movie llibrary using ./plex_trakt_sync.sh sync --sync=movies is working, at least as far as I know.

Syncing of the tv library using ./plex_trakt_sync.sh sync --sync=tv leads in my case to an error.
Keep in mind that I am using the beta agent of plex.

Attached you will find the error.log and the last lines of
last_update.log.

Seems to be the same error mentioned in #220.

@wandabastyle wandabastyle mentioned this pull request Apr 10, 2021
simonc56
simonc56 previously approved these changes Apr 12, 2021
@glensc
Copy link
Collaborator Author

glensc commented Apr 17, 2021

Merging this to master:

thank you all for testing!

@glensc glensc merged commit 72bb5ed into master Apr 17, 2021
@glensc glensc deleted the main branch April 17, 2021 08:35
@glensc glensc mentioned this pull request Apr 21, 2021
Repository owner locked as resolved and limited conversation to collaborators Jan 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants