-
Notifications
You must be signed in to change notification settings - Fork 191
Add Donetick #3630
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
base: master
Are you sure you want to change the base?
Add Donetick #3630
Conversation
|
Realised the previous implementation could be improved by implementing Additional Environment Variables list instead of specific JWT Secret so this way people can customise any aspect of the app: https://docs.donetick.com/getting-started/configration#configuration-via-environment-variables |
|
Was also wondering if we should use SQLite for simplicity or Postgres for scaling? How do you usually decide between these options? Or maybe you have a pattern for making these options as conditional templates? Or would that be too much maintenance with the different permutations multiplying? |
|
Hello, I'm sorry but this is completely out of standard with all other apps. You also assumed a lot of users preferences like host paths. Healthcheck didnt pass either :) |
|
Blocked by donetick/donetick#420 |
Sorry about that, I did check If you could add some more info (and maybe links to app examples) to make it easier to learn these patterns than randomly clicking around into other apps' directories, that would be super useful for new contributors! Anyway, thanks for picking it up! |
This is my first app so winging it a bit 😅 sorry if I misunderstood something and the structure isn't right!
Managed to get it running with:
Loads correctly:
Could also sign up and create a task that persisted after page reload: