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

#470 - added section on Pay Yourself First strategy #473

Merged

Conversation

RubenOlsen
Copy link
Member

A go at #470

@actual-github-bot actual-github-bot bot changed the title #470 - added section on Pay Yourself First strategy [WIP] #470 - added section on Pay Yourself First strategy Oct 14, 2024
Copy link

netlify bot commented Oct 14, 2024

Deploy Preview for actualbudget-website ready!

Name Link
🔨 Latest commit 64cb249
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget-website/deploys/677c2787403cd90008e0c849
😎 Deploy Preview https://deploy-preview-473.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
Member

youngcw commented Oct 18, 2024

I think it would be good to give a brief example of how to do this, not just what it is. Also my blog post from last year shows how I do this with templates if you want to link there.

@RubenOlsen
Copy link
Member Author

Good point! I see where you are coming from and I did think about adding a brief example.

However - I wanted to keep it as short as the Month Ahead strategy. For now, I'll add an example and also link to the blogpost. I'll see if I also can add a brief example in the month ahead paragraph.

@RubenOlsen RubenOlsen changed the title [WIP] #470 - added section on Pay Yourself First strategy #470 - added section on Pay Yourself First strategy Oct 23, 2024
@actual-github-bot actual-github-bot bot added the 🔍 Ready for review Someone needs to look into this. label Oct 23, 2024
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.

Minor nit for consistency

docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
@actual-github-bot actual-github-bot bot added ⚠️ Changes requested and removed 🔍 Ready for review Someone needs to look into this. labels Nov 3, 2024
@actual-github-bot actual-github-bot bot added 🔍 Ready for review Someone needs to look into this. and removed ⚠️ Changes requested labels Jan 3, 2025
@RubenOlsen
Copy link
Member Author

@matt-fidd roasting time again :-)

docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
docs/getting-started/envelope-budgeting.md Outdated Show resolved Hide resolved
@actual-github-bot actual-github-bot bot added ✅ Approved Used in conjunction with Merge at Release. This means that the PR has been approved but not merged. and removed 🔍 Ready for review Someone needs to look into this. labels Jan 6, 2025
@RubenOlsen RubenOlsen requested a review from matt-fidd January 6, 2025 18:59
@RubenOlsen RubenOlsen merged commit 2fa1d34 into actualbudget:master Jan 8, 2025
6 checks passed
@actual-github-bot actual-github-bot bot added ✨ Merged and removed ✅ Approved Used in conjunction with Merge at Release. This means that the PR has been approved but not merged. labels Jan 8, 2025
@RubenOlsen RubenOlsen deleted the 470-PayYourSelfFirstStrategy branch January 8, 2025 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants