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

Documentation: provide details on storage engine album naming #6812

Merged
merged 2 commits into from Feb 1, 2024

Conversation

mmomjian
Copy link
Contributor

Explain how the {{album}} variable is handled in file paths.

@@ -17,7 +17,7 @@ Year/Year-Month-Day/Filename.Extension
<img src={require('./img/storage-template.png').default} width="100%" title="Storage Template Setting" />

:::tip
By default, special characters will be converted to an HTML entity (for example, `&` -> `&amp;`). To prevent this, wrap the variable in an extra set of braces (for example, `{{{album}}}`). You can learn more about this [here](https://handlebarsjs.com/guide/expressions.html#html-escaping) and [here](https://github.com/immich-app/immich/issues/4917).
The `{{album}}` variable will expand to the name of the album to which the file was most recently added to. By default, special characters will be converted to an HTML entity (for example, `&` -> `&amp;`). To prevent this, wrap the variable in an extra set of braces (for example, `{{{album}}}`). You can learn more about this [here](https://handlebarsjs.com/guide/expressions.html#html-escaping) and [here](https://github.com/immich-app/immich/issues/4917).
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think it is "most recently added to". It is actually based on when the album was created. We might want to change this, but that is what it looks like it does right now.

image

@mmomjian
Copy link
Contributor Author

mmomjian commented Jan 31, 2024

Updated. I started a discussion in #6810 with a feature request to improve this handling, but for now I think we can document what the current behavior is.

@jrasm91 jrasm91 added the documentation Improvements or additions to documentation label Feb 1, 2024
@jrasm91
Copy link
Contributor

jrasm91 commented Feb 1, 2024

Perfect! Although that seems like an unrelated discussion you linked.

@jrasm91 jrasm91 merged commit 81cf653 into immich-app:main Feb 1, 2024
22 of 24 checks passed
@mmomjian
Copy link
Contributor Author

mmomjian commented Feb 1, 2024

Sorry, I meant #6810

@mmomjian mmomjian deleted the doc-albums branch March 31, 2024 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants