Skip to content

Conversation

@Convly
Copy link
Member

@Convly Convly commented Aug 1, 2023

What does it do?

  • Add the type definition for the entity service's findOne method

files?: unknown; // TODO
};

type SelectReadWrite<TContentTypeUID extends Common.UID.ContentType> = SelectRead<TContentTypeUID> &
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this for a later implementation?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep: create, update, etc..

Copy link
Contributor

@innerdvations innerdvations left a comment

Choose a reason for hiding this comment

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

I like it. We should be able to add another generic param to Select* in the future to handle the issue we discussed the other day with only including parameters supported by the method

@Convly
Copy link
Member Author

Convly commented Aug 1, 2023

[Waiting for #17519 to be merged, and then merging this one]

Base automatically changed from types/entity-services/base-type to features/types/entity-service August 1, 2023 14:57
@Convly Convly merged commit b5928b3 into features/types/entity-service Aug 1, 2023
@Convly Convly deleted the types/entity-services/find-one branch August 1, 2023 14:58
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