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 dynamic enum Tutorial #278

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

LukasBoll
Copy link
Contributor

closes #270

Copy link
Member

@sdirix sdirix 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!

Please add a small section at the end that this behavior can be generalized and extracted into React HOCs for cleaner code.

content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
@LukasBoll LukasBoll force-pushed the dynamicEnumTutorial branch from 3af8f33 to 99d4152 Compare March 4, 2024 15:41
@LukasBoll LukasBoll requested a review from sdirix March 4, 2024 15:44
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
/>
);
};
```
Copy link
Member

Choose a reason for hiding this comment

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

Please add the wrapping code for the renderers, i.e. withJsonFormsControlProps etc. here and in the other listings.

content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
content/docs/tutorial/dynamic-enum.mdx Outdated Show resolved Hide resolved
@LukasBoll LukasBoll force-pushed the dynamicEnumTutorial branch from d2d24bc to d6205ec Compare April 15, 2024 12:52
Co-authored-by: Stefan Dirix <[email protected]>
closes eclipsesource#270
@LukasBoll LukasBoll force-pushed the dynamicEnumTutorial branch from 817dd55 to 23fa126 Compare April 15, 2024 15:27
@LukasBoll LukasBoll requested a review from sdirix April 15, 2024 15:33
@LukasBoll
Copy link
Contributor Author

thank you, I applied the changes from your review!

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.

Add FAQ section or tutorial: How to handle dynamic enum elements
2 participants