[PR #11872] [MERGED] chore(donetick): add config entry for v0.1.73 #9806

Closed
opened 2026-02-26 17:31:48 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/11872
Author: @tomfrenzel
Created: 2/13/2026
Status: Merged
Merged: 2/13/2026
Merged by: @CrazyWolf13

Base: mainHead: donetick-0.1.73


📝 Commits (1)

  • 550793a fix(donetick): add config entry for v0.1.73

📊 Changes

1 file changed (+2 additions, -1 deletions)

View changed files

📝 ct/donetick.sh (+2 -1)

📄 Description

✍️ Description

This adds back a config entry that was previously removed because of preventing the app to start (#11804). The new dontick version now re added support of this entry so I updated the script to make sure it gets added again in case someone has already ran the update script for the previous version which removed the entry.

Maybe it's also an option to just remove the sed -i '/capacitor:\/\/localhost/d' /opt/donetick/config/selfhosted.yaml I prevously added and have users manaully re-add the capacitor://localhost CORS entry in case they need it.
Not sure which way to go is the best here 😄

Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/11872 **Author:** [@tomfrenzel](https://github.com/tomfrenzel) **Created:** 2/13/2026 **Status:** ✅ Merged **Merged:** 2/13/2026 **Merged by:** [@CrazyWolf13](https://github.com/CrazyWolf13) **Base:** `main` ← **Head:** `donetick-0.1.73` --- ### 📝 Commits (1) - [`550793a`](https://github.com/community-scripts/ProxmoxVE/commit/550793a8390e1a76911d9e3070b2ff623f4f4fdc) fix(donetick): add config entry for v0.1.73 ### 📊 Changes **1 file changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `ct/donetick.sh` (+2 -1) </details> ### 📄 Description ## ✍️ Description This adds back a config entry that was previously removed because of preventing the app to start (#11804). The [new dontick version](https://github.com/donetick/donetick/releases/tag/v0.1.73) now re added support of this entry so I updated the script to make sure it gets added again in case someone has already ran the update script for the previous version which removed the entry. Maybe it's also an option to just remove the `sed -i '/capacitor:\/\/localhost/d' /opt/donetick/config/selfhosted.yaml` I prevously added and have users manaully re-add the ``capacitor://localhost`` CORS entry in case they need it. Not sure which way to go is the best here 😄 ## ✅ Prerequisites (**X** in brackets) - [X] **Self-review completed** – Code follows project standards. - [X] **Tested thoroughly** – Changes work as expected. - [X] **No security risks** – No hardcoded secrets, unnecessary privilege escalations, or permission issues. --- ## 🛠️ Type of Change (**X** in brackets) - [ ] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [ ] ✨ **New feature** – Adds new, non-breaking functionality. - [ ] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. - [ ] 🆕 **New script** – A fully functional and tested script or script set. - [ ] 🌍 **Website update** – Changes to website-related JSON files or metadata. - [X] 🔧 **Refactoring / Code Cleanup** – Improves readability or maintainability without changing functionality. - [ ] 📝 **Documentation update** – Changes to `README`, `AppName.md`, `CONTRIBUTING.md`, or other docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 17:31:48 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxmoxVE#9806
No description provided.