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

Proposed by Ted Gould
Status: Merged
Approved by: Charles Kerr
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) Approve
PS Jenkins bot (community) continuous-integration Approve
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
33. By Ted Gould

Using the right app

34. By Ted Gould

Again

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
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
=== modified file 'service/dispatcher.c'
--- service/dispatcher.c 2013-09-27 20:58:55 +0000
+++ service/dispatcher.c 2013-10-01 17:39:44 +0000
@@ -137,6 +137,12 @@
137137
138/* TODO: Make these come from registrations, but this works for now */138/* TODO: Make these come from registrations, but this works for now */
139url_type_t url_types[] = {139url_type_t url_types[] = {
140 /* Alarms */
141 {
142 .regex_patern = "^alarm:///",
143 .regex_object = NULL,
144 .app_id = "ubuntu-clock-app"
145 },
140 /* Address Book */146 /* Address Book */
141 {147 {
142 .regex_patern = "^addressbook:///",148 .regex_patern = "^addressbook:///",

Subscribers

People subscribed via source and target branches