Ticket #328 (reopened Bugs)

Opened 20 months ago

Last modified 6 months ago

Gmail notification doesn't show up

Reported by: deity Owned by: grim
Milestone: Guifications2.17 Component: Guifications2
Version: Guifications2.14 Severity: major
Keywords: Cc:

Description

I am not sure whether it is a missing feature or a bug. I seems to recall reading somewhere that only pop e-mail notification would show up??? But anyway, When I configure GAIM/Pidgin to connect to my Google Talk account via its Jabber server, I can set GAIM/Pidgin up so that it display new e-mail notification. It comes in the form of a yellow box in the main contact windows, and also it would flash in the taskbar (not system tray) (WinXP) if the "flash window when IM are received" preference is set for GAIM/Pidgin. However, it would be extremely helpful if Gunifications would work with e-mail notification via Jabber, because at the moment, Gunifications seems to just ignore it completely

Change History

Changed 13 months ago by jbailey

  • pending set to 1

We actually do listen for the proper events for e-mail received and act accordingly; however, not all themes implement e-mail notifications (our support of this event is relatively new compared to most of our existing themes), and you may need to explicitly enable notifications for mail received in the preferences window for Guifications.

Changed 12 months ago by jbailey

  • version set to Guifications2.14
  • severity set to major
  • milestone set to Guifications2.17

An additional wrinkle to note here is that if you have multiple new emails, we don't have a notification for that. If you have only a single new mail, then we will show a notification as expected, provided your loaded theme supports email notifications and you have email notifications enabled in the Guifications preferences as I noted in the previous comment.

It looks like we may need to add yet another event for multiple emails or figure out how to shoehorn it in to our existing event setup.

Changed 12 months ago by jbailey

For reference, the relevant code from gf_event.c is (the comment is mine, added here for clarification):

static void *
gf_event_email(PurpleConnection *gc, const char *subject, const char *from,
                const char *to, const char *url)
{
    gf_event_common("new-email", gc->account,
            purple_find_buddy(gc->account, from),
            NULL, from, NULL, PURPLE_CBFLAGS_NONE, NULL, subject);
    return real_notify_email(gc, subject, from, to, url);
}

static void *
gf_event_emails(PurpleConnection *gc, size_t count, gboolean detailed,
                const char **subject, const char **from,
                const char **to, const char **url)
{
    if (count == 1 && subject && *subject)
        return gf_event_email(gc, *subject, *from, *to, *url);
 /* Here is where the problem lies--we don't do a notification on multiple
  * email messages.  We need to either figure out a way to make the existing
  * single-mail notification be good enough or add yet another event. */
    return real_notify_emails(gc, count, detailed, subject, from, to, url);
}

Changed 11 months ago by trac-robot

  • status changed from new to closed
  • pending changed from 1 to 0

This ticket was closed automatically by the system. It was previously set to a Pending status and hasn't been updated within 21 days.

Changed 10 months ago by jbailey

  • status changed from closed to reopened

This ticket should actually not have closed. I must have forgotten to unset the "pending" flag.

Changed 6 months ago by sorin

Any status change on this issue?

Note: See TracTickets for help on using tickets.