Skip to content

Conversation

JamesDAdams
Copy link

No description provided.

@Copilot Copilot AI review requested due to automatic review settings September 28, 2025 19:00
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Komga integration functionality to the invites plugin, allowing automatic creation of Komga user accounts when invites are shared. The implementation includes configuration settings for Komga connection details and a new method to create user accounts via the Komga API.

  • Added Komga configuration settings including URI, API key, default password, roles, and library IDs
  • Implemented automatic Komga account creation when sharing invites
  • Created a dedicated method to handle Komga API communication with proper error handling

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
api/plugins/invites/plugin.php Added Komga settings UI, integration logic, and account creation method
api/plugins/invites/config.php Added default configuration values for Komga settings

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JamesDAdams JamesDAdams changed the title create komga account when user use invite create komga $ Nextcloud account when user use invite and an option for add the user to plex home Oct 2, 2025
$this->setLoggerChannel('Plex')->warning('Plex User already has access');
$this->setAPIResponse('error', 'Plex User already has access', 409);
return false;
$this->setAPIResponse('success', 'Plex User already has access', 200);
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I edit this because if the user who use the invitation have already acces on the plex server he stuck on at the stage where he gives his Plex username.

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