Error for growl notifications about foxtrick HOT 8 CLOSED

Comment #1 originally posted by larswortel on 2008-12-16T09:09:40.000Z:

Update: fixing the error above was easy: the run function of foxtrickalert.js
requested two parameters (page, doc) whereas it only gets one from foxtrick.js
because it's a run-on-every-page function (namely doc). This is fixed in 33c53b4 r159...
but unfortunately the notifications still do not work.

from foxtrick.

Comment #2 originally posted by larswortel on 2008-12-16T09:17:59.000Z:

Spoke too soon: just got a notification :) I guess my fix was sufficient after all.

Status: Fixed

from foxtrick.

Comment #3 originally posted by tiziano.m on 2008-12-16T10:24:05.000Z:

I reopen it because there is a bug: notifications don't work if the entries in the
news aren't 3. As stated before, i should stop writing code at 2 o'clock in the
night. I'll work on it.

Status: Started

from foxtrick.

Comment #4 originally posted by tiziano.m on 2008-12-16T11:53:44.000Z:

The bug for lines<3 is fixed. Now there are 2 smaller bugs:
if you receive 2 news at the same time, just one is shown.
when you open the page the first time, if there is a notification to be shown, you'll
get it not immediately but at first refresh (after 30sec)

from foxtrick.

Comment #5 originally posted by tiziano.m on 2008-12-16T12:32:43.000Z:

Also playsound seems not working

from foxtrick.

Comment #6 originally posted by tiziano.m on 2008-12-17T23:42:21.000Z:

playsound fixed (thanks kolmis). Still not working:
-if you get 2 news at the same time only last one is shown
-there is a delay between news is shown and the notification pop

from foxtrick.

Comment #7 originally posted by larswortel on 2009-01-15T19:06:27.000Z:

Finally fixed now in 9df28b4 r880

from foxtrick.

Comment #8 originally posted by larswortel on 2009-01-15T19:06:52.000Z:

<empty>

Status: Fixed

from foxtrick.