I noticed that the API key environment variable name isn't actually hard-coded:
-- config.lua
local api_key = os.getenv(M.api_key_env_var_name)
I think it would be nice if the currently required environment variable, OBSIDIAN_REST_API_KEY, was instead a default that the user can change should they wish. I propose an optional configuration option that the user can pass during plugin setup.
I can add this in a separate PR once I am done working on #27 .
I noticed that the API key environment variable name isn't actually hard-coded:
I think it would be nice if the currently required environment variable,
OBSIDIAN_REST_API_KEY, was instead a default that the user can change should they wish. I propose an optional configuration option that the user can pass during plugin setup.I can add this in a separate PR once I am done working on #27 .