[EventGrid] Small doc updates for id parameter#15835
Merged
lmazuel merged 1 commit intoAzure:masterfrom Dec 30, 2020
ellismg:ellismg/event-grid-doc-id-note
Merged
[EventGrid] Small doc updates for id parameter#15835lmazuel merged 1 commit intoAzure:masterfrom ellismg:ellismg/event-grid-doc-id-note
id parameter#15835lmazuel merged 1 commit intoAzure:masterfrom
ellismg:ellismg/event-grid-doc-id-note
Conversation
- For our model types, we've made `id` optional and when unspecified, a random UUID is used. Clarify this behavior in the docs (in our user studies most participants felt like this was the behavior, but it would have been helpful to see it explicitly called out). - `EventGridEvent` still some slightly incorrect documentation, due to some bad copy/pasting from `CloudEvent`. There is no `source` property in an `EventGridEvent`.
lmazuel
approved these changes
Dec 30, 2020
12 tasks
rakshith91
pushed a commit
to rakshith91/azure-sdk-for-python
that referenced
this pull request
Jan 8, 2021
- For our model types, we've made `id` optional and when unspecified, a random UUID is used. Clarify this behavior in the docs (in our user studies most participants felt like this was the behavior, but it would have been helpful to see it explicitly called out). - `EventGridEvent` still some slightly incorrect documentation, due to some bad copy/pasting from `CloudEvent`. There is no `source` property in an `EventGridEvent`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
For our model types, we've made
idoptional and when unspecified, arandom UUID is used. Clarify this behavior in the docs (in our user
studies most participants felt like this was the behavior, but it would
have been helpful to see it explicitly called out).
EventGridEventstill some slightly incorrect documentation, due tosome bad copy/pasting from
CloudEvent. There is nosourcepropertyin an
EventGridEvent.