Skip to content

Managed contacts #2

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 3 commits into from
Dec 14, 2017
Merged

Managed contacts #2

merged 3 commits into from
Dec 14, 2017

Conversation

Dorthu
Copy link
Collaborator

@Dorthu Dorthu commented Dec 13, 2017

This is a migrated linode/manager#2674

Dorthu pushed a commit to Dorthu/linode-api-docs that referenced this pull request Dec 13, 2017
Added ID to contacts and services; removed dupe
Copy link
Collaborator

@yoshizzle yoshizzle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please check this against API code to make sure I have it 100% correct

email:
description: The email address for the contact.
type: String
phone1:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be "primary" and in a "phone" envelope

Copy link
Collaborator Author

@Dorthu Dorthu Dec 13, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should actaully be phone_primary

phone1:
description: A primary phone number for the contact.
type: String
phone2:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be "secondary" and in a "phone" envelope

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should actaully be phone_secondary

-X POST -d '{
"name": "John Q. Public",
"email": "[email protected]",
"phone1": "215-555-3000",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"phone": {
"primary": "215-555-3000",
"secondary": "215-555-3001"
}

-X PUT -d '{
"name": "John Q. Public, Jr.",
"email": "[email protected]",
"phone1": "215-555-4000",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as above

type: String
description: >
The email address of the contact person.
phone1:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

primary

type: String
description: >
If we need further information regarding a service, we may call this contact if a phone number is provided.
phone2:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

secondary

@Dorthu Dorthu changed the title [DO NOT MERGE] Managed contacts Managed contacts Dec 13, 2017
Josh Sager and others added 2 commits December 13, 2017 10:09
Added managed to prebuild.js
Created Manged Contacts object
Created endpoints for managed services
Included both GET, POST, and PUT endpoints (consolidating two API-based
tasks into one docs PR)
"name": "John Q. Public",
"email": "[email protected]",
"phone_primary": "215-555-3000",
"phoen_secondary": "215-555-3001",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doh!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you used to call me on my cell phoen

@SzySteve SzySteve merged commit 2af04ec into linode:develop Dec 14, 2017
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
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.

3 participants