Skip to content

Remove photos support#246

Merged
lukasdotcom merged 4 commits into
mainfrom
photos
Jun 30, 2025
Merged

Remove photos support#246
lukasdotcom merged 4 commits into
mainfrom
photos

Conversation

@lukasdotcom
Copy link
Copy Markdown
Member

@lukasdotcom lukasdotcom commented Jun 27, 2025

Due to the new api for google photos (https://developers.google.com/photos/picker/guides/get-started-picker) not being very good for importing photos to nextcloud importing photos should be removed.

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Copy link
Copy Markdown
Member

@julien-nc julien-nc left a comment

Choose a reason for hiding this comment

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

Looks good, works fine.

Comment thread README.md Outdated

## Limitations

This app can not migrate google photos files due to limitations in the Google Photos API making it too complex for end users.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
This app can not migrate google photos files due to limitations in the Google Photos API making it too complex for end users.
This app can not migrate Google photos files due to limitations in the Google Photos API making it too complex for end users.

Maybe add a link to the change or the relevant part of their doc?

Copy link
Copy Markdown
Member Author

@lukasdotcom lukasdotcom Jun 30, 2025

Choose a reason for hiding this comment

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

I added a link to the notice about the api removal: https://developers.google.com/photos/support/updates#affected-scopes-methods

Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
Signed-off-by: Lukas Schaefer <lukas@lschaefer.xyz>
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.

2 participants