Skip to content

Conversation

carlos-villavicencio-adsk
Copy link
Contributor

  • Clarify some parts of Entity Fields section.
  • Clarify some parts of Combining Related Queries section.
  • Update doc navigability with links.

Copy link
Contributor

@shaynacohen shaynacohen left a comment

Choose a reason for hiding this comment

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

Suggestions left, but otherwise looks good from CXD pending the staged review from @NorberMV

dictionary::
When you do a :meth:`~shotgun_api3.Shotgun.find` or a :meth:`~shotgun_api3.Shotgun.create` call
that returns a field of type **entity** or **multi-entity** (for example the 'assets' column on Shot),
the entities are returned in a standard dictionary::
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
the entities are returned in a standard dictionary::
the entities are returned in a standard dictionary:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the feedback. I think this one is not applicable because next line is an indented code line and if I understand correctly, belongs to the literal block format.

Copy link
Contributor

Choose a reason for hiding this comment

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

Gotcha, thanks for clarifying!

Co-authored-by: Shayna Duguid <shayna.cohen@shotgunsoftware.com>
@carlos-villavicencio-adsk carlos-villavicencio-adsk merged commit 86f64f3 into master Jan 31, 2023
@carlos-villavicencio-adsk carlos-villavicencio-adsk deleted the SG-21729_update-docs-entity-fields branch January 31, 2023 21:51
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.

2 participants