Skip to content
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

Add entity-provider functions in the core-data documentation #51142

Open
wants to merge 3 commits into
base: trunk
Choose a base branch
from

Conversation

aurooba
Copy link
Member

@aurooba aurooba commented May 31, 2023

What?

This PR updates the documentation for the core-data package to include the hooks and providers in the core-data/src/entity-provider.js file.

Why?

Folks outside of just core can use functions like useEntityProp and they should be documented and available in the handbook

How?

So I had to make a small code change in order for this to work well. the EntityProvider function was being exported as a default function. As far as I could tell, there's no way for me to provide an alternative name so docgen would name this function default in the markdown file, which is no good.

So I switched it so that it's no longer a default export and just a normal named export from the file. From what I can tell, this doesn't cause any issues anywhere, but I'd love for someone to confirm this!

@aurooba aurooba requested a review from nerrad as a code owner May 31, 2023 21:50
@aurooba aurooba added the [Package] Core data /packages/core-data label Jun 8, 2023
@Mamaduka Mamaduka requested a review from adamziel June 13, 2023 06:02
@aurooba aurooba added the [Type] Developer Documentation Documentation for developers label Jun 16, 2023
@bacoords
Copy link
Contributor

+1 to getting this reviewed and merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Core data /packages/core-data [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants