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

Cleanup: Skip creating empty default server #1608

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

glensc
Copy link
Collaborator

@glensc glensc commented Oct 9, 2023

No need to convert servers config on the initial run, i.e. when PLEX_SERVER from .env is not initialized.

/app # plextraktsync
WARNING  plextraktsync without command is deprecated. Executing "plextraktsync sync"                                                     
INFO     PlexTraktSync [unknown]                                                                                                         
WARNING  Added default server to /app/config/servers.yml                                                                                 
Please enter your Plex username or e-mail: ^C
Aborted!

/app # cat /app/config/servers.yml
servers:
  default:
    token: null
    urls:
    - null
    - null
    id: null
    config: null
/app # 

@glensc glensc self-assigned this Oct 9, 2023
@glensc glensc marked this pull request as draft December 4, 2023 10:25
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