This repository has been archived by the owner on Nov 1, 2023. It is now read-only.
Replies: 2 comments 1 reply
-
|
Beta Was this translation helpful? Give feedback.
1 reply
-
We should also start versioning the templates at this point. And we can assume from this point that a template with no version is a jinja template. We could optionally add a language field to the template too. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
We currently accept notification templates using the jinja templating language. Jinja isn't supported in C# so we must migrate to a new templating engine. Scriban was chosen because it's inspired by jinja so the templates are very similar.
The current state
Notification templates are currently being processed and rendered by the python code. Once #2486 merges, we'll be able to adapt our existing jinja templates to be compatible with scriban. The template is not updated in the database, only in memory. This is intended to be a 'compatibility mode' until the migration is complete.
The desired state
We only want to accept scriban templates. We only store scriban templates in our database.
How do we get there?
QueueFileChanges
.QueueFileChanges
as the default when deploying OneFuzz.Potential issues
Beta Was this translation helpful? Give feedback.
All reactions