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

Better debugging for other templated fields #155

Closed
bobemoe opened this issue Oct 12, 2024 · 4 comments
Closed

Better debugging for other templated fields #155

bobemoe opened this issue Oct 12, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@bobemoe
Copy link

bobemoe commented Oct 12, 2024

Is your feature request related to a problem? Please describe.

I am developing complex functionality in a templated header (example in #153) but debugging outputs are limited compared to a templated message.

When the app starts, its seems to load and parse any message templates on startup and report any syntax errors as fatal errors and refuse to start. Additionally when an alert is sent there is debug output showing the final rendered content generate from the template. Both these are very useful in debugging but seem to be missing from templated headers (and maybe other places templates are used?)

Describe the solution you'd like

Identical functionality that of the message template as described above.

@0x2142
Copy link
Owner

0x2142 commented Oct 14, 2024

That's a fair request - I'll see what I can do!

@0x2142 0x2142 self-assigned this Oct 14, 2024
@0x2142 0x2142 added the enhancement New feature or request label Oct 14, 2024
@0x2142 0x2142 added this to the v0.4.0 milestone Oct 14, 2024
0x2142 added a commit that referenced this issue Oct 23, 2024
@0x2142
Copy link
Owner

0x2142 commented Oct 23, 2024

Just an FYI that I've merged a lot of this into the latest dev build 😊

Edit - I see that I missed part of the early config validation where the templates are checked to ensure they are parse-able... I'll include this in the next round

@freefd
Copy link
Contributor

freefd commented Nov 18, 2024

@bobemoe is it still relevant or can it be closed as solved?

@bobemoe
Copy link
Author

bobemoe commented Nov 18, 2024

I haven't tested it yet but looking at the commits, it looks like you implemented what I requested! And a nice bit of code tidying/reuse too :) I'll open a new issue if I find anything unexpected when I do try it out. Cheers

@bobemoe bobemoe closed this as completed Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants