-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Recurring transactions are not being created #1583
Comments
Yeah I was afraid this might happen. So Laravel only supports cron jobs which need to fire every minute. It's literally the least useful way of doing cron jobs. Even IFTTT calling special URL's would be more useful. What I am going to do for the next release is see if I can add as many "pseudo cron jobs" as possible. I know that Firefly III can handle it if you call the recurring transaction routine twice in one day. It will not break or do things double. What I would like to introduce is a useful way of doing cron jobs (besides what Laravel offers):
That would be a lot more useful than what there is now. In fact, that last one could be called daily by IFTTT if you have an account there. It's just that I focused on the feature itself, and not yet on the cron job. So stay tuned! |
So long story short I can not yet reliably use recurring transactions? That's sad because I was really looking forward to that feature. But I guess it won't kill me to create some transactions manually like I always did until there is a solid solution. |
There is no reason why it should not work on Windows. I can Google several solutions to get a cron job going on windows that people say will work. It will take at least 24hrs to get it going, this is something with Laravel counting to the next time it should run. Until Laravel feels its time for the job nothing will happen. I believe this is 02:00am. I'm just saying I will add even easier methods 😉 |
I have already successfully setup the command to run every minute. It hasn't created any transactions though. I'll wait until tomorrow to check whether Laravell is just 24hours off its schedule :D |
In the new version you should be able to set the daily |
I've seen it already. I guess l'll upgrade this weekend and the check whether it works. Thanks a lot for the improvement |
Hi, I made the upgrade yesterday. Through a misconfigured recurring transaction (set to daily instead of monthly) I already found out that it's working now :D |
Sorry, I just need to ask. I'm trying to follow the https://firefly-iii.readthedocs.io/en/latest/installation/cronjob.html using IFTTT, however, when I do the call to the address in in it https://mydomainname.com/cron/run/profileidthing it never works. It keeps failing, and when I look into my folders I can't see a folder called cron, or run, have I done the link wrong? |
https://demo.firefly-iii.org/cron/run/57eb79c7989ed0d53f3efc1535b8c9b8 It should say "fired successfully" or "The recurring transaction cron job did not fire.". Does it say either of these things or do you get another error? Oh, and the URL is a fake URL but it should work just the same. |
Yeah, I was more wondering if it should be https://mydomain.org/57eb79c7989ed0d53f3efc1535b8c9b8 I get two different errors: "Unable to make web request. Your server returned a 404" So I wonder if my link is wrong towards the path of the cron. Or maybe it is something when you isntall through Softaculos again? Like it was with my other issue. |
What is the URL of the login page? Could you share everything after the |
Yeah, for sure. the full address to my login is: https://fin.postnow.eu/public, in my IFTTT I'm using this address: http://fin.postnow.eu/public/cron/run/code |
I also tried having http://fin.postnow.eu/cron/run/code |
Should I give you my code so you can test? And then just generate a new one? |
This should be the URL: https://fin.postnow.eu/public/cron/run/code What happens when you go there in the browser? |
" Unfortunately, this error was not recoverable. Firefly III broke. The error is: Trying to get property 'data' of non-object |
I saw it. Can you check on your server, the content of the /storage/logs directory? |
Yeah, I have several files, two from today (or three), anything special I should look for, or should I just post the whole thing here? |
If you follow the instructions to enable debug mode and then only visit the page with /cron/run the log that will be created can tell me exactly what is going on. Just send it to me over email, because it could contain private data. I think it's a bug but I want to be sure. |
Ah, okay! Will do. |
Sent! :) thanks for helping me look into it. |
It seems to be a bug. Do you have multiple users in your installation of Firefly III? I've pushed a fix for the next release. You can check it out in the commit above my reply on GitHub. If you want to, you can replace the line yourself and it should work for you as well. |
We, or well, I have set-up a few accounts for some friends, so we are sharing the costs and such, since we all liked the idea of the app! :) I hope that is fine! And once again, thank you! |
Of course, that's perfectly fine. That was the idea of making it "multi-user". I just had to make sure that you knew about it. You can also fix this issue by making everybody login and visit their /profile page. |
Ah, well, I updated the line and will test if it works now! :) |
Sorry... Seems like I got a new issue now. :( "Argument 2 passed to FireflyIII\Factory\TransactionFactory::findAccount() must be of the type integer or null, string given, called in /home/postagzd/fin.postnow.eu/app/Factory/TransactionFactory.php on line 123" Do you want me to get a full debug again? |
Yes please. I need to know which code is the bad code. |
Sent! Sorry for the trouble. |
No problem, just pushed another fix. |
Wow, that speed. Testing now. |
That seems to have fixed it! My applet run (I couldn't get my cronjob to work no matter what I did, so decided to try the IFTTT suggestion you have in the docs). And now the applet ran, and I no longer have the blue box saying cron-job hasn't ever run over my re-curring. So now I will test adding one for tomorrow and se if that get's added! Thanks a lot for the help! |
No problem 👍 @skuzzle sorry for spamming your mailbox 😉 |
No problem. I'm, watching every issue anyways :) |
Hi,
I'm still running 4.7.5 on a Windows Server.
As mentioned before, I have set up the Windows Task Scheduler to perform the
php artisan schedule:run
command every minute (is it really necessary to call the command every minute?)Now it seems that firefly still does not create any recurring transactions. I added multiple RTs for the 1st of the month on 2 firefly user accounts but none has been created.
When running the command manually I get the output
When running from task scheduler I can't see the output but I can see that the commands exits with code
0
There is also no hint in the log files (I've configured level 'warning' for logging).
The text was updated successfully, but these errors were encountered: