bug
avatar betweenbrain
betweenbrain
10 Nov 2013

joomla/joomla-cms#2448 is closed, and yet http://issues.joomla.org/tracker/joomla-cms/2448 reports that it is open.

avatar betweenbrain betweenbrain - open - 10 Nov 2013
avatar betweenbrain betweenbrain - open - 10 Nov 2013
avatar mbabker
mbabker - comment - 10 Nov 2013

Gah, I was hoping that was fixed (it seemed that it had been at a glance).

So the general issue is that it looks like the GitHub API isn't always picking up on activities in commits to trigger events.

avatar b2z
b2z - comment - 10 Nov 2013

That's disappointing :disappointed:

avatar elkuku
elkuku - comment - 10 Nov 2013

Bad stuff :(
I "think" we should run a cron and check periodically for updates - as an addition to the GitHub hooks - just in case ;)
Would that be an option ?

avatar mbabker
mbabker - comment - 10 Nov 2013

I'd support that. Would also be good for those eventual GitHub downtimes.

On Sunday, November 10, 2013, Nikolai Plath wrote:

Bad stuff :(
I "think" we should run a cron and check periodically for updates - as an
addition to the GitHub hooks - just in case ;)
Would that be an option ?


Reply to this email directly or view it on GitHub#175 (comment)
.

avatar betweenbrain
betweenbrain - comment - 10 Nov 2013

:+1:

avatar b2z
b2z - comment - 10 Nov 2013

I thought about cron also. How often should it run - every hour or less may be?

avatar betweenbrain
betweenbrain - comment - 10 Nov 2013

Would this be a true, server-side cron, or traffic based pseudo-cron? If traffic based, could it be once every hour unless we exceed a certain traffic level?

avatar mbabker
mbabker - comment - 10 Nov 2013

So we already have a script to do these updates, add a true server-side cron job triggering that, have a nice day :-)

avatar mbabker
mbabker - comment - 10 Nov 2013

OK, so note, that script is only doing labels. But the update command can be expanded on or use the get command.

avatar betweenbrain
betweenbrain - comment - 10 Nov 2013

So we already have a script to do these updates, add a true server-side cron job triggering that, have a nice day :-)

:smile: Cool

avatar elkuku elkuku - reference | - 18 Nov 13
avatar b2z
b2z - comment - 23 Jan 2014

What is the current status?

avatar b2z
b2z - comment - 11 Feb 2014

@mbabker can we close it?

avatar mbabker
mbabker - comment - 11 Feb 2014

Ya, we can. Cron jobs I think are working well enough for us now.

avatar mbabker mbabker - close - 11 Feb 2014

Add a Comment

Login with GitHub to post a comment