Skip to content

Conversation

@alecmerdler
Copy link
Contributor

@alecmerdler alecmerdler commented May 14, 2018

Description

Introduces firehoseFor factory function, which returns a Firehose-like component that accepts a render callback and provides type autocompletion based on the resources provided.

Changes

  • Add types to existing firehose module
  • Add firehoseFor factory function and tests
  • Convert all <Firehose> instances in TypeScript components to firehoseFor

Addresses https://jira.coreos.com/browse/CONSOLE-327
Previously https://github.com/coreos-inc/bridge/pull/2189

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 14, 2018
@openshift openshift deleted a comment from coreos-ui May 14, 2018
@openshift openshift deleted a comment from coreos-ui May 14, 2018
@alecmerdler alecmerdler force-pushed the CONSOLE-327 branch 2 times, most recently from 3dd5ee2 to 4552878 Compare May 24, 2018 01:51
@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 11, 2018
@openshift openshift deleted a comment from stevekuznetsov Jun 27, 2018
@openshift openshift deleted a comment from openshift-bot Jun 27, 2018
@openshift openshift deleted a comment from openshift-ci-robot Jun 27, 2018
@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 27, 2018
@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 28, 2018
@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 28, 2018
@openshift openshift deleted a comment from openshift-ci-robot Jun 28, 2018
@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 1, 2018
@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 3, 2018
@openshift-bot
Copy link
Contributor

@alecmerdler: PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 26, 2018
@alecmerdler alecmerdler closed this Aug 6, 2018
jeff-phillips-18 pushed a commit to jeff-phillips-18/console that referenced this pull request Oct 15, 2019
Replace arrow markers with ConnectorArrow component
TimothyAsirJeyasing pushed a commit to TimothyAsirJeyasing/console that referenced this pull request Aug 3, 2022
fix: Add number in front of Storage System in Status Card
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants