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

Next version #540

Open
wants to merge 240 commits into
base: main
Choose a base branch
from
Open

Next version #540

wants to merge 240 commits into from

Conversation

simulot
Copy link
Owner

@simulot simulot commented Nov 24, 2024

A complete rewrite of Immich-go

simulot and others added 30 commits July 16, 2024 04:00
* edit release.md

* Merge remote-tracking branch 'origin/next' into github-ci/cd
* remove dead code

* Fix: Add HTTP Status to API trace #381
Fixes the following error message:

```
cannot unmarshal number into Go struct field
ExifInfo.Assets.items.exifInfo.fileSizeInByte of type int
```

This is caused by a video in immich that is larger than can fit into an
int32.
If a user is skimming the documentation it can be easy to miss
(e.g. #473), add a small hint to make it more obvious.
- Updated `Browse` and `passTwo` methods in `LocalAssetBrowser` to return `AssetGroup` instead of `LocalAssetFile`.
- Implemented logic to handle different types of asset groups (e.g., motion photos, single images, videos).
- Added album management for asset groups.
- Introduced `AssetGroup` struct in `group.go` to represent linked assets.
- Updated tests to validate and handle `AssetGroup` instead of individual assets.
- Removed redundant album management code from `assetFromFile` method.
- Updated `ReadMetadata`, `metadataFromExiftool`, and `metadataFromDirectRead` functions to return `*metadata.Metadata` and `error` instead of setting `la.Metadata` directly.
- Adjusted logic to handle the returned metadata and error appropriately.
- Add functionality to list the flags and arguments used in the command to provide more context in the logs.
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.

3 participants