Merge lp:~ted/url-dispatcher/alarm-url into lp:url-dispatcher/13.10

Proposed by Ted Gould on 2013-10-01
Status: Merged
Approved by: Charles Kerr on 2013-10-02
Approved revision: 34
Merged at revision: 32
Proposed branch: lp:~ted/url-dispatcher/alarm-url
Merge into: lp:url-dispatcher/13.10
Diff against target: 16 lines (+6/-0)
1 file modified
service/dispatcher.c (+6/-0)
To merge this branch: bzr merge lp:~ted/url-dispatcher/alarm-url
Reviewer Review Type Date Requested Status
Charles Kerr (community) 2013-10-01 Approve on 2013-10-02
PS Jenkins bot (community) continuous-integration Approve on 2013-10-01
Review via email: mp+188665@code.launchpad.net

Commit message

Adding the alarm URL pattern

Description of the change

Alarms alarms alarms

To post a comment you must log in.
lp:~ted/url-dispatcher/alarm-url updated on 2013-10-01
33. By Ted Gould on 2013-10-01

Using the right app

34. By Ted Gould on 2013-10-01

Again

Charles Kerr (charlesk) wrote :

Looks right.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'service/dispatcher.c'
2--- service/dispatcher.c 2013-09-27 20:58:55 +0000
3+++ service/dispatcher.c 2013-10-01 17:39:44 +0000
4@@ -137,6 +137,12 @@
5
6 /* TODO: Make these come from registrations, but this works for now */
7 url_type_t url_types[] = {
8+ /* Alarms */
9+ {
10+ .regex_patern = "^alarm:///",
11+ .regex_object = NULL,
12+ .app_id = "ubuntu-clock-app"
13+ },
14 /* Address Book */
15 {
16 .regex_patern = "^addressbook:///",

Subscribers

People subscribed via source and target branches