Skip to content

Conversation

@karaolidis
Copy link
Contributor

@karaolidis karaolidis commented Sep 3, 2025

Description

Resolves #7406 and updates some obsidian configurations to the latest upstream defaults.

Checklist

  • Change is backwards compatible.

  • Code formatted with nix fmt or
    nix-shell -p treefmt nixfmt deadnix keep-sorted --run treefmt.

  • Code tested through nix run .#tests -- test-all or
    nix-shell --pure tests -A run.all.

  • Test cases updated/added. See example.

  • Commit messages are formatted like

    {component}: {description}
    
    {long description}
    

Signed-off-by: Nikolaos Karaolidis <[email protected]>
Signed-off-by: Nikolaos Karaolidis <[email protected]>
Signed-off-by: Nikolaos Karaolidis <[email protected]>
@karaolidis karaolidis marked this pull request as draft September 3, 2025 17:04
@karaolidis karaolidis marked this pull request as ready for review September 3, 2025 17:21
template = (pkgs.formats.json { }).generate "obsidian.json" {
vaults = builtins.listToAttrs (
builtins.map (vault: {
name = builtins.substring 0 16 (builtins.hashString "md5" vault.target);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am pretty sure that the name of each vault is a random string, so this would cause a duplicated vault to appear if a user creates one outside home-manager and then switches to a declarative definition. Not sure if there's much we can do about this...

Signed-off-by: Nikolaos Karaolidis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Obsidian: Doesn't open up last used vault on startup

1 participant