8000 Add status_callback_url to PhoneNumber.update() by mgp · Pull Request #89 · twilio/twilio-python · GitHub
[go: up one dir, main page]

Skip to content

Add status_callback_url to PhoneNumber.update() #89

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 5, 2013
Merged

Add status_callback_url to PhoneNumber.update() #89

merged 1 commit into from
Jan 5, 2013

Conversation

mgp
Copy link
Contributor
@mgp mgp commented Oct 22, 2012

The StatusCallback property of phone numbers is unique in that it 
specifies a URL, but the property name does not end in "Url" like 
VoiceUrl, VoiceFallbackUrl, SmsUrl, and SmsFallbackUrl do. As a
convenience, the purchase method of PhoneNumbers automatically
assigns any status_callback_url argument to the StatusCallback
property. This change adds the same convenience to the update
method of PhoneNumber.

The StatusCallback property of phone numbers is unique in that it 
specifies a URL, but the property name does not end in "Url" like 
VoiceUrl, VoiceFallbackUrl, SmsUrl, and SmsFallbackUrl do. As a 
convenience, the purchase method of PhoneNumbers automatically 
assigns any status_callback_url argument to the StatusCallback 
property. This change adds the same convenience to the update 
method of PhoneNumber.
@gbin
Copy link
Contributor
gbin commented Jan 4, 2013

+1

1 similar comment
@ryanhorn
Copy link
Contributor
ryanhorn commented Jan 5, 2013

+1

gbin added a commit that referenced this pull request Jan 5, 2013
Add status_callback_url to PhoneNumber.update()
@gbin gbin merged commit d446c40 into twilio:master Jan 5, 2013
@kevinburke
Copy link
Contributor

hm. If you specify neither what is the API behavior? i believe we would send StatusCallback=None to the API which may get rejected because None is not a valid URL.

eq19 pushed a commit to VirtualNetworks/parser that referenced this pull request May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0