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

Fixing #490 Improve bank sync instructions #603

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

Conversation

RubenOlsen
Copy link
Member

  • Moved relevant information from gocardless.md to bank-syn.md and rewrote this to be general.
  • Fixed header level from 3 to 2 in gocardless.md and simplefin.md.

Issue link: #490

- Moved relevant information from gocardless.md to bank-syn.md and rewrote this to be general.
- Fixed header level from 3 to 2 in gocardless.md and simplefin.md.
@actual-github-bot actual-github-bot bot changed the title Fixing #490 Improve bank sync instructions [WIP] Fixing #490 Improve bank sync instructions Jan 6, 2025
@RubenOlsen RubenOlsen requested a review from matt-fidd January 6, 2025 17:44
Copy link

netlify bot commented Jan 6, 2025

Deploy Preview for actualbudget-website ready!

Name Link
🔨 Latest commit 0485bf2
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget-website/deploys/677c4f5697221e000838febd
😎 Deploy Preview https://deploy-preview-603.www.actualbudget.org
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@RubenOlsen RubenOlsen changed the title [WIP] Fixing #490 Improve bank sync instructions Fixing #490 Improve bank sync instructions Jan 6, 2025
@actual-github-bot actual-github-bot bot added 🔍 Ready for review Someone needs to look into this. and removed 🚧 WIP labels Jan 6, 2025
docs/advanced/bank-sync.md Fixed Show fixed Hide fixed

This comment has been minimized.

@RubenOlsen RubenOlsen changed the title Fixing #490 Improve bank sync instructions [WIP] Fixing #490 Improve bank sync instructions Jan 6, 2025
@actual-github-bot actual-github-bot bot added 🚧 WIP and removed 🔍 Ready for review Someone needs to look into this. labels Jan 6, 2025
@RubenOlsen RubenOlsen changed the title [WIP] Fixing #490 Improve bank sync instructions Fixing #490 Improve bank sync instructions Jan 6, 2025
@actual-github-bot actual-github-bot bot added 🔍 Ready for review Someone needs to look into this. and removed 🚧 WIP labels Jan 6, 2025
Copy link
Contributor

@matt-fidd matt-fidd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of comments on a quick skim through, I'll get a proper review in when I get a chance.

docs/transactions/bank-sync/index.md Outdated Show resolved Hide resolved
docs/transactions/bank-sync/index.md Outdated Show resolved Hide resolved

## Does Actual Sync Automatically With Your Bank?

At this moment, it is not yet possible for Actual to automatically sync with your bank. You need to do this manually by going to _All Accounts_ and pressing _Bank Sync_.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a thought, since the bank sync button is on every linked account page as well, would it make sense to adjust this line a touch more to something like "... by going to your account/All Accounts and pressing Bank Sync" to help indicate that you can do either (batch or individual)?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is a good point.

And while I'm at it - add that it's best to do it on the All Accounts page since that will save on the number of syncs rather than on each account. Both GoCardless and SimpleFIN have a maximum number of available syncs over X time.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that sounds good to me. could help with some of the confusion about rate limits per provider

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. Please add in that you can also sync individual accounts from the respective account page. Its probably good to recommend the all accounts page though.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That is a good point.

And while I'm at it - add that it's best to do it on the All Accounts page since that will save on the number of syncs rather than on each account. Both GoCardless and SimpleFIN have a maximum number of available syncs over X time.

This isn't strictly true, it won't make any difference to GoCardless rate limits. I don't think it matters to SimpleFin either because their rate limits are pretty lenient, although it will be slower.


- The integration only works if you are using actual-server.

- The Secrets and Keys are stored in your Actual installed version so it is highly recommended to turn on End to End encryption and create a strong passphrase to encrypt your files.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems not quite right. The keys are stored in the server so it shouldn't matter that much if encryption is enabled or not.


- The Secrets and Keys are stored in your Actual installed version so it is highly recommended to turn on End to End encryption and create a strong passphrase to encrypt your files.

- You will need to add a config file to your installation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't true. You don't need one for simplefin at least.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nor GoCardless


## Does Actual Sync Automatically With Your Bank?

At this moment, it is not yet possible for Actual to automatically sync with your bank. You need to do this manually by going to _All Accounts_ and pressing _Bank Sync_.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. Please add in that you can also sync individual accounts from the respective account page. Its probably good to recommend the all accounts page though.

The following process may be more helpful:

1. Set up your account in Actual specifying a correct opening account balance at a recent date.
2. Link the account one of the supported providers (GoCardLess or SimpleFIN).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably add how to do the link process via the account menu.

@@ -14,7 +14,7 @@ but also addresses some of its limitations.
One key difference is that traditional envelope budgeting often overlooks fixed expenses like rent, mortgage,
utility bills, and other regular payments. These fixed costs are crucial and must be accounted for to have a
complete picture of your finances. Actual Budget ensures that every fixed or variable expense is
considered. This comprehensive approach makes Actual Budget much more effective than the traditional cash-stuffing method.
considered. This comprehensive approach makes AB much more effective than the traditional cash-stuffing method.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this file diff from the other recent PR? It all looks the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔍 Ready for review Someone needs to look into this.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants