Closed rui-nar closed 1 month ago
We had a failure in the tests - but it was just a slightly timing sensitive test and re-run passed.
Note that the failure in the docker image build phase herer https://github.com/gilesknap/gphotos-sync/actions/runs/9368991609 is benign - its because the PR actions run with your permissions and can't see my docker credentials.
So all good. Merging.
Thanks!
@rui-nar let me know if you want me to release this or wait for more updates!
You can release it. I have some other code tweaks but I'll open an "issue" first to explain my approach since they change some aspects of the way of working. We'll see if you like it or not. If you do I'll package and do a PR accordingly.
In any case, these were my first PRs and I'm really happy to contribute. I'm building a more extensive workflow to work with Immich so maybe more PRs will come, not sure yet.
On Tue, Jun 4, 2024, 18:26 Giles Knap @.***> wrote:
@rui-nar https://github.com/rui-nar let me know if you want me to release this or wait for more updates!
— Reply to this email directly, view it on GitHub https://github.com/gilesknap/gphotos-sync/pull/492#issuecomment-2147945141, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACP6EX7FBX4QRTWJ4AW2UHDZFXTEXAVCNFSM6AAAAABIYUXD76VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBXHE2DKMJUGE . You are receiving this because you were mentioned.Message ID: @.***>
Immich: that sounds interesting!
added ability to separate the folders for albums and shared albums using command line arguments. It'll default to the albm folder if no shared album folder path is given.