Skip to content

[desktop][photos] Parse favorited key from google takeout.#10828

Closed
AswinAsok wants to merge 2 commits into
ente-io:mainfrom
AswinAsok:feat/favorite-takeout-files
Closed

[desktop][photos] Parse favorited key from google takeout.#10828
AswinAsok wants to merge 2 commits into
ente-io:mainfrom
AswinAsok:feat/favorite-takeout-files

Conversation

@AswinAsok
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e6b7be21c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment on lines +679 to +680
const hiddenCollectionIDs = new Set(
(await savedHiddenCollections()).map((c) => c.id),
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Avoid favoriting new hidden-album imports

When a user imports a Google Takeout folder into a newly created hidden album, createHiddenAlbum returns a remote-only collection and the local saved collections are not refreshed until postUploadAction() runs after this handler, so savedHiddenCollections() does not contain that new hidden collection yet. The files uploaded to that hidden collection therefore pass this filter and get added to the visible Favorites collection, leaking hidden imports; use the upload/target collections' visibility in addition to saved state before calling addToFavoritesCollection.

Useful? React with 👍 / 👎.

@AswinAsok AswinAsok closed this Jun 5, 2026
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