Reminders trigger at the time of the event and not before by default
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | Canonical System Image |
High
|
Pat McGowan | ||
| | Ubuntu Calendar App |
High
|
Kunal Parmar | ||
| | indicator-datetime (Ubuntu) |
High
|
Charles Kerr | ||
| | qtorganizer5-eds (Ubuntu) |
Undecided
|
Renato Araujo Oliveira Filho | ||
Bug Description
Sync of google calendar. The Google calendar setting is set by default to notify 10 min prior to all meetings.
Is it possible to get this default notification time from google with the sync?
or can we add a setting for a default notification time in the calendar app?
Related branches
- PS Jenkins bot: Approve (continuous-integration) on 2015-04-06
- Ted Gould (community): Approve on 2015-04-06
-
Diff: 1791 lines (+834/-269)33 files modifieddebian/control (+11/-8)
include/datetime/alarm-queue-simple.h (+6/-12)
include/datetime/alarm-queue.h (+1/-1)
include/datetime/appointment.h (+19/-4)
include/datetime/clock-mock.h (+1/-1)
include/datetime/clock.h (+1/-1)
include/datetime/date-time.h (+5/-1)
include/datetime/planner-snooze.h (+1/-2)
include/datetime/snap.h (+2/-1)
include/datetime/wakeup-timer-mainloop.h (+2/-2)
include/datetime/wakeup-timer-powerd.h (+2/-2)
include/datetime/wakeup-timer.h (+1/-1)
src/actions-live.cpp (+13/-6)
src/actions.cpp (+2/-2)
src/alarm-queue-simple.cpp (+140/-109)
src/appointment.cpp (+9/-3)
src/date-time.cpp (+31/-9)
src/engine-eds.cpp (+153/-54)
src/main.cpp (+7/-5)
src/planner-snooze.cpp (+10/-6)
src/snap.cpp (+9/-7)
tests/CMakeLists.txt (+33/-4)
tests/manual-test-snap.cpp (+5/-5)
tests/print-to.h (+45/-0)
tests/run-eds-test.sh (+57/-0)
tests/test-alarm-queue.cpp (+10/-7)
tests/test-eds-valarms-config-files/.config/evolution/sources/system-proxy.source (+21/-0)
tests/test-eds-valarms-config-files/.local/share/evolution/calendar/system/calendar.ics (+47/-0)
tests/test-eds-valarms.cpp (+101/-0)
tests/test-live-actions.cpp (+0/-6)
tests/test-snap.cpp (+10/-10)
tests/timezone-mock.h (+1/-0)
tests/wakeup-timer-mock.h (+78/-0)
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration) on 2015-05-22
- Ubuntu Calendar Developers: Pending requested 2015-04-29
-
Diff: 60 lines (+11/-10)2 files modifiedNewEvent.qml (+10/-9)
tests/autopilot/calendar_app/tests/test_new_event.py (+1/-1)
| summary: |
- Reminders trigger at the time of the event and not before + Reminders trigger at the time of the event and not before by default |
| Sebastien Bacher (seb128) wrote : | #1 |
| Charles Kerr (charlesk) wrote : | #2 |
This looks like it might be two related but separate issues.
1. handling the notification time for events imported from google calendar
2. displaying triggers at the correct time, rather than at the time of the event. At first glance, this second one appears to be an indicator-
| Sebastien Bacher (seb128) wrote : | #3 |
Pat, I'm adding "canonical-
| Pat McGowan (pat-mcgowan) wrote : | #4 |
In fact we want to include calendar by default
| Changed in canonical-devices-system-image: | |
| importance: | Undecided → High |
| milestone: | none → ww09-2015 |
| status: | New → Confirmed |
| Pat McGowan (pat-mcgowan) wrote : | #5 |
assigning bill as I am not sure where the fix(es) resides
| Changed in canonical-devices-system-image: | |
| assignee: | nobody → Bill Filler (bfiller) |
| Changed in canonical-devices-system-image: | |
| milestone: | ww09-2015 → ww13-2015 |
| Kunal Parmar (pkunal-parmar) wrote : | #6 |
"Is it possible to get this default notification time from google with the sync?"
I think this needs to be handled at EDS plugin end. When EDS syncs the events it needs to check google's notification setting and modify the event in local storage.
| Changed in qtorganizer5-eds (Ubuntu): | |
| assignee: | nobody → Renato Araujo Oliveira Filho (renatofilho) |
I can confirm that the event reminders are included on calendar event, and they can be retrieved/updated by QtOrganizer API.
The information that we have is:
BEGIN:VALARM^M
TRIGGER;
ACTION:DISPLAY^M
DESCRIPTION:This is an event reminder^M
X-EVOLUTION-
END:VALARM^M
| Changed in qtorganizer5-eds (Ubuntu): | |
| status: | New → Invalid |
| Bill Filler (bfiller) wrote : | #8 |
indicator-datetime needs to be modified to read this information out of the event and display the notification at the appropriate time
| Changed in ubuntu-calendar-app: | |
| status: | New → Invalid |
| Changed in indicator-datetime (Ubuntu): | |
| assignee: | nobody → Charles Kerr (charlesk) |
| importance: | Undecided → High |
| Changed in canonical-devices-system-image: | |
| assignee: | Bill Filler (bfiller) → Thomas Strehl (strehl-t) |
| tags: | added: bq |
| Changed in indicator-datetime (Ubuntu): | |
| status: | New → Triaged |
| Charles Kerr (charlesk) wrote : | #9 |
Adding ubuntu-calendar-app as also affected. Its "New Events" page for creating calendar events defaults to "On Event" rather than, say, 15 minutes before.
| Changed in indicator-datetime (Ubuntu): | |
| status: | Triaged → In Progress |
| Changed in ubuntu-calendar-app: | |
| status: | Invalid → New |
| no longer affects: | ubuntu-calendar-app (Ubuntu) |
| Changed in ubuntu-calendar-app: | |
| status: | New → Confirmed |
| Launchpad Janitor (janitor) wrote : | #10 |
This bug was fixed in the package indicator-datetime - 13.10.0+
---------------
indicator-datetime (13.10.
[ Charles Kerr ]
* Improve valarm support to honor calendar events' valarm triggers.
(LP: #1419001)
-- CI Train Bot <email address hidden> Mon, 06 Apr 2015 23:35:46 +0000
| Changed in indicator-datetime (Ubuntu): | |
| status: | In Progress → Fix Released |
| Pat McGowan (pat-mcgowan) wrote : | #11 |
@popey can you get this scheduled
| Changed in ubuntu-calendar-app: | |
| assignee: | nobody → Alan Pope ๎ฟ (popey) |
| importance: | Undecided → High |
| Changed in canonical-devices-system-image: | |
| assignee: | Thomas Strehl (strehl-t) → Pat McGowan (pat-mcgowan) |
| milestone: | ww13-2015 → ww17-2015 |
| Changed in ubuntu-calendar-app: | |
| milestone: | none → 2015-04-30 |
| Changed in ubuntu-calendar-app: | |
| status: | Confirmed → In Progress |
| assignee: | Alan Pope ๎ฟ (popey) → Kunal Parmar (pkunal-parmar) |
| Changed in canonical-devices-system-image: | |
| milestone: | ww17-2015 → ww21-2015 |
Fix committed into lp:ubuntu-calendar-app at revision 650, scheduled for release in ubuntu-
| Changed in ubuntu-calendar-app: | |
| status: | In Progress → Fix Committed |
| Changed in ubuntu-calendar-app: | |
| status: | Fix Committed → Fix Released |
doesn't seem to work with google calendar.
| zulucloud (zulucloud) wrote : | #14 |
Hmmpf... doesn't seem to work on my bq.
Test 1:
Updated everything possible, including calendar app. No reboot, no restarting of any app. Created a new event (creation time appr. 15:55) to start at 16:10, set reminder to 5 minutes. Alarm went off at 16:10.
Test 2:
Switched off bq; switched on again. Created a new event to start at 16:35 (creation time appr. 16:15), leaving reminder at default / 15 minutes. Alarm went off at 16:35.
Let me know if i can help in any way..
| zulucloud (zulucloud) wrote : Re: [Bug 1419001] Re: Reminders trigger at the time of the event and not before by default | #15 |
Hmmpf... doesn't seem to work on my bq.
Test 1:
Updated everything possible, including calendar app. No reboot, no
restarting of any app. Created a new event (creation time appr. 15:55)
to start at 16:10, set reminder to 5 minutes. Alarm went off at 16:10.
Test 2:
Switched off bq; switched on again. Created a new event to start at
16:35 (creation time appr. 16:15), leaving reminder at default / 15
minutes. Alarm went off at 16:35.
Let me know if i can help in any way..
Am 27.05.2015 um 13:52 schrieb Alan Pope ๎ฟ:
> ** Changed in: ubuntu-calendar-app
> Status: Fix Committed => Fix Released
>
when inserting a new date into LOCAL calendar, having german timezone, entering no data, but saving this date, afterwards it is moved two hours before the entered time.
| Changed in canonical-devices-system-image: | |
| status: | Confirmed → Fix Released |
| Changed in canonical-devices-system-image: | |
| milestone: | ww21-2015 → ww22-2015 |


confirmed on rtm233