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

[WIP] First go at #452 (Simple FIN Bank Sync Article modification) #453

Closed

Conversation

RubenOlsen
Copy link
Member

Fixes for #452

Copy link

netlify bot commented Sep 17, 2024

Deploy Preview for actualbudget-website ready!

Name Link
🔨 Latest commit 60e17f7
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget-website/deploys/66e9d5c7b50ce7000854690a
😎 Deploy Preview https://deploy-preview-453.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.

@youngcw
Copy link
Contributor

youngcw commented Sep 22, 2024

Instead could we just update the simplefin docs to move out of experimental? So fix #452 and #454

matt-fidd added a commit that referenced this pull request Oct 3, 2024
…#2 (#459)

- removed simplefin from experimental tab
- added mention of simplefin in bank-sync
- split bank-sync into gocardless and simplefin
- created dropdown for bank-sync
- proofreaded punctuation & image alignment in gocardless and simplefin
guides

Fixes #452,
#453 and fixes
#454

I'm not sure why #458 got
closed, so I'm making a new one here.

---------

Co-authored-by: youngcw <calebyoung94@gmail.com>
Co-authored-by: Matt Fiddaman <github@m.fiddaman.uk>
@psybers
Copy link
Contributor

psybers commented Oct 6, 2024

@youngcw This PR, and the associated issue, are now stale as the SimpleFIN feature is no longer experimental and the docs are already updated.

@RubenOlsen
Copy link
Member Author

Killing OF this PR as it is outdated.

@RubenOlsen RubenOlsen closed this Oct 6, 2024
@RubenOlsen RubenOlsen deleted the SimpleFINBankSync-Issue452 branch October 14, 2024 16:26
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.

3 participants