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

UI: Component sharing framework #3259

Closed
8 tasks done
JBWatenbergScality opened this issue Apr 2, 2021 · 1 comment
Closed
8 tasks done

UI: Component sharing framework #3259

JBWatenbergScality opened this issue Apr 2, 2021 · 1 comment
Assignees
Labels
priority:urgent Any issue we should jump in as soon as possible topic:ui UI-related issues

Comments

@JBWatenbergScality
Copy link
Contributor

JBWatenbergScality commented Apr 2, 2021

Component:

UI

Why this is needed:

Some solutions UI would need to be able to reuse components defined in Metalk8s UI.

What should be done:

As a start we want to export the following components :

  • AlertProvider
  • useAlerts
  • useNodesAlert
  • useVolumesAlert
  • useServiceAlert
  • useNetworkAlert
  • getNodesCountQuery
  • getVolumesCountQuery

Test plan:

Integration test with a solution UI
Is this chunk available to be imported independently than the UI ?
Does it bundles its required dependencies correctly ?
Is it resilient to peer dependencies upgrades ?

@JBWatenbergScality JBWatenbergScality added the topic:ui UI-related issues label Apr 2, 2021
@JBWatenbergScality JBWatenbergScality self-assigned this Apr 2, 2021
@thomasdanan thomasdanan added the priority:urgent Any issue we should jump in as soon as possible label Apr 13, 2021
@thomasdanan thomasdanan added this to the MetalK8s 2.9.0 milestone Apr 13, 2021
@ChengYanJin
Copy link
Contributor

Done in PR #3295

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:urgent Any issue we should jump in as soon as possible topic:ui UI-related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants