Skip to content
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

Actual Budget Role #404

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Actual Budget Role #404

wants to merge 18 commits into from

Conversation

Norin-Radd
Copy link

@Norin-Radd Norin-Radd commented Dec 6, 2024

Description

Actual Budget is a super fast and privacy-focused app for managing your finances. At its heart is the well proven and much loved Envelope Budgeting methodology.
You own your data and can do whatever you want with it. Featuring multi-device sync, optional end-to-end encryption and so much more.

https://hub.docker.com/r/actualbudget/actual-server

https://actualbudget.org/

https://actualbudget.org/docs/

I tested this through my sandbox repo and it's working and it populates my data folder after last commits.

Deleted the last part of template cause it was optional

@maximuskowalski
Copy link
Collaborator

You would need to edit sandbox.yml to make this a role and test it properly.

Basically, switch your sandbox install to your fork, fetch and pull and switch to the branch you have used to create the role and you should be able to install like any sandbox role.

@Norin-Radd
Copy link
Author

You would need to edit sandbox.yml to make this a role and test it properly.

Basically, switch your sandbox install to your fork, fetch and pull and switch to the branch you have used to create the role and you should be able to install like any sandbox role.

I'm sorry, but I'll need further instructions on how to do that via Discord. I tried to follow the documents, but I couldn't get there.

@Norin-Radd
Copy link
Author

You would need to edit sandbox.yml to make this a role and test it properly.

Basically, switch your sandbox install to your fork, fetch and pull and switch to the branch you have used to create the role and you should be able to install like any sandbox role.

Hello, thanks for the help.

I already tested it on my server running saltbox, please feel free to test further and evaluate the pull request.

Formatting / spacing
Formatting, blank line at end of file
Extra blank line
@maximuskowalski
Copy link
Collaborator

Looking at the recommended compose it looks like your config location
- "{{ actualbudget_paths_location }}:/opt/data"
Should actually be
- "{{ actualbudget_paths_location }}:/data"

Which then uses the sub folders /data/server-files and /data/user-files

It looks like the config file if supplied is meant to be config.json which means this is likely wrong if you are trying to supply a config file but I can't see any pre-config happening anyway.:
actualbudget_config_location: "{{ actualbudget_paths_location }}/configuration.yaml"

It doesn't appear that this ENV var is required:
- "/run/udev:/run/udev:ro"
There are ENV options listed here https://actualbudget.org/docs/config/

You will also need to supply the link to your DOCS PR before this will be accepted.

Did the Pull Request template fail to populate when you made it?
You should have seen something like this:
https://github.com/saltyorg/Sandbox/blob/master/.github/PULL_REQUEST_TEMPLATE.md

@Norin-Radd
Copy link
Author

saltyorg/docs#323

Docs pull request link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants