-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Breaking: Actual Budget Script (HTTPS / DB Migration / New Structure) - Read Description #2496
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dev port removed
CrazyWolf13
reviewed
Feb 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise lgtm.
CrazyWolf13
previously requested changes
Feb 20, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line 124 Shouldn't there be https?
I cannot really select it as it's out of range for the normal review.
tremor021
approved these changes
Feb 20, 2025
michelroegl-brunner
approved these changes
Feb 20, 2025
csaller
pushed a commit
to csaller/ProxmoxVE
that referenced
this pull request
Feb 22, 2025
… - Read Description (community-scripts#2496)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
breaking change
A change that is not backward compatible
update script
A change that updates a script
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
ActualBudget Update & Migration Guide
📌 Important: Backup Your Data First
Before updating, export your Actual Budget data to avoid data loss. Follow these steps:
For detailed instructions, see the Backup & Restore Guide.
🚀 Key Changes in the New Version
1️⃣ Database Changes
password
column inauth
./opt/actualbudget-data/migrate/
.2️⃣ File Structure Overhaul
/opt/actualbudget-data/
./opt/actualbudget-data/server-files/
/opt/actualbudget-data/user-files/
/opt/actualbudget-data/migrations/
/opt/actualbudget-data/migrate/
/opt/actualbudget-data/config/
.env
is now stored in/opt/actualbudget-data/
.3️⃣ New HTTPS Support
4️⃣ If Issues Persist, Reinstall in a New LXC
bash -c "$(wget -qLO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/actualbudget.sh)"
🔐 Enabling HTTPS
Generate a Self-Signed Certificate
Configure HTTPS
Update
.env
:Restart service:
Renew Certificate Manually
🎯 Summary
✅ Backup before updating (see Backup Guide)
✅ HTTPS supported but needs manual renewal
✅ If update fails, migrate manually or reinstall LXC
🔗 Related PR / Discussion / Issue
Link: #74 #1623 #807 #1707 #1354 #2490
✅ Prerequisites
Before this PR can be reviewed, the following must be completed:
🛠️ Type of Change
Select all that apply: