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

refactor(core): allow checklist component to accept objects as a prop #7058

Merged
merged 1 commit into from
May 23, 2019

Conversation

Jammy-Louie
Copy link

make checklist component more flexible by allowing the user to be able to specify a list of strings or a list of objects, similar to react-select

@Jammy-Louie
Copy link
Author

Currently looking to refactor Notifications into React.
Wanted to reuse this component for the the refactor effort

@anotherchrisberry
Copy link
Contributor

LGTM

@anotherchrisberry anotherchrisberry merged commit 50f23fc into spinnaker:master May 23, 2019
@Jammy-Louie Jammy-Louie deleted the update-checklist branch May 23, 2019 15:15
@christopherthielen
Copy link
Contributor

nice 👍

anotherchrisberry added a commit that referenced this pull request May 28, 2019
…#7066)

* chore(core): Bump version to 0.0.373

89e4e78 fix(runJob/kubernetes): reliably display logs (#7060)
50f23fc refactor(core): allow checklist component to accept objects as a prop (#7058)
0eaae9a refactor(core): expose clusterService in react injector (#7043)
8dd1417 fix(core): Build triggers: Properly render large number of jobs (#7056)
4975233 fix(artifacts/helm): support regex/spel in version (#7033)

* chore(docker): Bump version to 0.0.41

59259a9 fix(docker): Fixed imageId parsing and digest support (#7053)

* chore(titus): Bump version to 0.0.98

50f23fc refactor(core): allow checklist component to accept objects as a prop (#7058)
59259a9 fix(docker): Fixed imageId parsing and digest support (#7053)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants