Bug 35098 - Need an error type for indicating lack of support for emergency numbers
Summary: Need an error type for indicating lack of support for emergency numbers
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: tp-spec (show other bugs)
Version: git master
Hardware: Other All
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2011-03-07 15:28 UTC by Jonathon Jongsma
Modified: 2011-03-09 05:15 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
patch (1.18 KB, patch)
2011-03-07 15:31 UTC, Jonathon Jongsma
Details | Splinter Review
Add EmergencyCallsNotSupported error (1.42 KB, patch)
2011-03-08 07:16 UTC, Jonathon Jongsma
Details | Splinter Review
Add EmergencyCallsNotSupported error (1.43 KB, patch)
2011-03-08 07:30 UTC, Jonathon Jongsma
Details | Splinter Review
Add EmergencyCallsNotSupported error (1.43 KB, patch)
2011-03-08 08:08 UTC, Jonathon Jongsma
Details | Splinter Review

Description Jonathon Jongsma 2011-03-07 15:28:49 UTC
When a user attempts to call a number that is an emergency number (e.g. '911' in the USA), there should be an error for indicating that a call failed because the connection manager does not have support for dialling emergency numbers.

I propose to call it 'org.freedesktop.Telepathy.Error.EmergencyCallsNotSupported'
Comment 1 Jonathon Jongsma 2011-03-07 15:31:54 UTC
Created attachment 44214 [details] [review]
patch
Comment 2 Danielle Madeley 2011-03-07 16:54:18 UTC
Review of attachment 44214 [details] [review]:

::: spec/errors.xml
@@ +551,3 @@
+      not support dialling emergency numbers.
+    </tp:docstring>
+    <tp:rationale>

tp:rationale should be inside tp:docstring.
Comment 3 Sjoerd Simons 2011-03-08 06:21:08 UTC
Comment on attachment 44214 [details] [review]
patch

It's not necessarily the CM, your voip to pstn provider might not support this. Would be nice to clarify the rationale in some way to say that a lot of providers do not support this. One example being Google voice (http://www.google.com/googlevoice/legal-notices.html first item)
Comment 4 Jonathon Jongsma 2011-03-08 07:16:50 UTC
Created attachment 44233 [details] [review]
Add EmergencyCallsNotSupported error

https://bugs.freedesktop.org/show_bug.cgi?id=35100
Comment 5 Jonathon Jongsma 2011-03-08 07:30:34 UTC
Created attachment 44234 [details] [review]
Add EmergencyCallsNotSupported error
Comment 6 Jonathon Jongsma 2011-03-08 08:08:04 UTC
Created attachment 44240 [details] [review]
Add EmergencyCallsNotSupported error
Comment 7 Jonathon Jongsma 2011-03-08 08:08:39 UTC
Comment on attachment 44234 [details] [review]
Add EmergencyCallsNotSupported error

one more time, with correct name this time.
Comment 8 Sjoerd Simons 2011-03-08 18:04:01 UTC
Looks good to me, wjt might want to have a quick look (it's late and i might have missed something :p)
Comment 9 Sjoerd Simons 2011-03-09 05:15:25 UTC
Merged, thanks!.  A fresh spec has just released and a new telepathy-glib release should be out soon


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.