Skip to content

Conversation

@lukesneeringer
Copy link
Contributor

No description provided.

@lukesneeringer lukesneeringer requested a review from a team as a code owner July 19, 2021 19:38
@google-cla google-cla bot added the cla: yes label Jul 19, 2021
- The operations (verbs) each resource provides, relying as much as possible on
the standard operations.

### Resources

Choose a reason for hiding this comment

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

A couple of immediate questions...

  1. How would a "join" query look that faults in resources that cross multiple domains? Should this definition include some notion of composition?
  2. I'm going to assume that a resource includes multiple "sub resources" (e.g. order header plus order lines with additional reference/inclusion-of other resources (e.g. shipping and billing address for customer). How will this fold in here?


Resource-oriented APIs emphasize resources (data model) over the operations
performed on those resources (functionality). A typical resource-oriented API
exposes a large number of resources with a small number of operations on each
Copy link
Contributor Author

Choose a reason for hiding this comment

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

an arbitrary number

verbs (usually `POST`) and define the custom verb in the URI. See AIP-136 for
more detail.

Services **should** prefer standard operations over custom operations; the
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the spot to include the guidance about including all the standard operations unless you have a reason.

Copy link

@mkistler mkistler left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants