From cb618cd46cb9a265065d53975ff27857f47694a2 Mon Sep 17 00:00:00 2001
From: "Oleg \"Zmey!\" Vasiliev" <ovasiliev@proton.me>
Date: Thu, 26 Oct 2023 21:29:35 +0300
Subject: [PATCH] Template fixes

---
 defaults/main.yml       | 15 ++++++++-------
 templates/config.yml.j2 |  2 +-
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index d55222e..fbeafbc 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -1,10 +1,10 @@
 ---
 
 # Reference: https://vikunja.io/docs/config-options/
+# <something> - is not a real values, it's placeholders: https://github.com/go-vikunja/api/issues/76
+# So set them to desired values or comment out to use Vikunja defaults
 
 vikunja_config:
-  # <something> - is not a real values, it's placeholders: https://github.com/go-vikunja/api/issues/76
-  # So set them to desired values or comment out to use Vikunja defaults
   service:
     # This token is used to verify issued JWT tokens.
     # Default is a random token which will be generated at each startup of vikunja.
@@ -50,12 +50,12 @@ vikunja_config:
     # Whether totp is enabled. In most cases you want to leave that enabled.
     enabletotp: true
     # If not empty, enables logging of crashes and unhandled errors in sentry.
-    sentrydsn: ''
+    sentrydsn: ""
     # If not empty, this will enable `/test/{table}` endpoints which allow to put any content in the database.
     # Used to reset the db before frontend tests. Because this is quite a dangerous feature allowing for lots of harm,
     # each request made to this endpoint needs to provide an `Authorization: <token>` header with the token from below. <br/>
     # **You should never use this unless you know exactly what you're doing**
-    testingtoken: ''
+    testingtoken: ""
     # If enabled, vikunja will send an email to everyone who is either assigned to a task or created it when a task reminder
     # is due.
     enableemailreminders: true
@@ -102,9 +102,9 @@ vikunja_config:
     # Whether to enable redis or not
     enabled: false
     # The host of the redis server including its port.
-    host: 'localhost:6379'
+    host: "localhost:6379"
     # The password used to authenticate against the redis server
-    password: ''
+    password: ""
     # 0 means default database
     db: 0
 
@@ -329,7 +329,8 @@ vikunja_config:
     # If set to true will send an email every day with all overdue tasks at a configured time.
     overdue_tasks_reminders_enabled: true
     # When to send the overdue task reminder email.
-    overdue_tasks_reminders_time: 9:00
+    # Quotes are important here: the preventing parsing value as time and convert it to minutes on output to yaml.
+    overdue_tasks_reminders_time: "9:00"
     # The id of the default project. Make sure users actually have access to this project when setting this value.
     default_project_id: 0
     # Start of the week for the user. `0` is sunday, `1` is monday and so on.
diff --git a/templates/config.yml.j2 b/templates/config.yml.j2
index beef28c..f52c60c 100644
--- a/templates/config.yml.j2
+++ b/templates/config.yml.j2
@@ -1,4 +1,4 @@
 ---
 {{ ansible_managed }}
 
-{{ vikunja_config | to_yaml }}
+{{ vikunja_config | to_nice_yaml(indent=2, sort_keys=False) }}