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

CUI-156 Add new admin components directory with microfrontend container #689

Closed
wants to merge 8 commits into from

Conversation

MichalGorskiOnegini
Copy link
Contributor

No description provided.

@MichalGorskiOnegini MichalGorskiOnegini requested a review from a team as a code owner April 26, 2024 07:23

The component is based on the following design: [LINK](https://www.figma.com/file/tZLMF6dw6LOSkmEhlV3rkF/UI-Patterns?type=design&node-id=2107%3A16959&mode=design&t=9UWjpVqhIeYZOXXR-1)

# Props
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd see example usage here ;P. WDYT? Worth to add?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's quite specific so IMO it's sufficient how it is

/*
* FIXME: This component is meant to be moved to a separate UCL package when it's finished.
*/
export const MicrofrontendContainer: ForwardRefRenderFunction<HTMLDivElement, Props> = ({
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Make it experimental, as it may change when a new form component is ready.

Copy link

sonarcloud bot commented May 16, 2024

@MichalGorskiOnegini
Copy link
Contributor Author

After discussing with @pawel-napieracz we decided it will be handled correctly in a separate ticket: https://onewelcome.atlassian.net/browse/UCL-478

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants