Skip to content

Add important note about api key permissions in immich #40183

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: current
Choose a base branch
from
Draft

Add important note about api key permissions in immich #40183

wants to merge 1 commit into from
+4 −1

Conversation

mib1185
Copy link
Contributor

@mib1185 mib1185 commented Jul 28, 2025

Proposed change

as mentioned in #39668 (comment)

with immich 1.136.0 the immich-app/immich#20113 has been overhauled, so this section needs a bit more love in a coming up PR

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@Copilot Copilot AI review requested due to automatic review settings July 28, 2025 21:24
@home-assistant home-assistant bot added current This PR goes into the current branch in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels Jul 28, 2025
@home-assistant
Copy link

It seems that this PR is targeted against an incorrect branch since it has a parent PR on one of our codebases. Documentation that needs to be updated for an upcoming release should target the next branch. Please change the target branch of this PR to next and rebase if needed.

@netlify Netlify
Copy link

netlify bot commented Jul 28, 2025 •
edited
Loading

✅ Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 02ba829
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6887eaa808e2d300080a01f7
😎 Deploy Preview https://deploy-preview-40183--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@mib1185 mib1185 marked this pull request as draft July 28, 2025 21:25
Copilot
Copilot AI reviewed Jul 28, 2025
View reviewed changes
Copy link
Contributor

@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 an important note about API key permissions for the Immich integration due to changes in Immich version 1.136.0. The change informs users that they now need to enable "all" permissions when creating an API key, replacing the previous more granular permission requirements.

Key changes:

  • Adds a prominent warning about API key permission requirements for Immich v1.136.0+
  • Notes that detailed permissions table will be provided in future updates

@mib1185
Copy link
Contributor Author

mib1185 commented Jul 28, 2025

With immich-app/immich#20250 (which will be release in immich server 1.137) this important note is not needed anymore, or could be changed, that in 1.136 the all is needed and beginning with 1.137 a detailed permissions table could be provided

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@mib1185 mib1185

Labels
current This PR goes into the current branch in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mib1185