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

Adding Create Schedule From Transaction #303

Merged
merged 3 commits into from
Feb 27, 2024
Merged

Conversation

xentara1
Copy link
Contributor

As discussed..
actualbudget/actual#2222

Copy link

netlify bot commented Jan 13, 2024

Deploy Preview for actualbudget-website ready!

Name Link
🔨 Latest commit 7649c9d
🔍 Latest deploy log https://app.netlify.com/sites/actualbudget-website/deploys/65d5d5c3c86f200008dfaa90
😎 Deploy Preview https://deploy-preview-303.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 Jan 23, 2024

@xentara1 Can you look into why there are build errors? My guess is that there is some bad markdown syntax

@youngcw
Copy link
Member

youngcw commented Jan 25, 2024

Ok, one last thing. Would it make more sense to put this in the schedules page instead of the bulk actions page?

@xentara1
Copy link
Contributor Author

xentara1 commented Jan 25, 2024 via email

@youngcw
Copy link
Member

youngcw commented Jan 25, 2024

I think it makes more sense in the schedules page since most of the time it will be used with a single transaction not multiple

@xentara1
Copy link
Contributor Author

Moved

@youngcw
Copy link
Member

youngcw commented Feb 21, 2024

Thanks. Ill merge this in once we know that the current bug is fixed and the feature is working.

@youngcw youngcw merged commit 6f42b9f into actualbudget:master Feb 27, 2024
6 checks passed
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