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

feat(core/presentation): support JSX for validationMessage in FormFields #7083

Merged

Conversation

christopherthielen
Copy link
Contributor

e.g.:
<FormikFormField validationMessage={<Spinner />} />

Also adds 2 new validationStatus codes: success and none

e.g.:
<FormikFormField validationMessage={<Spinner />} />
@christopherthielen christopherthielen merged commit e6d1586 into spinnaker:master Jun 4, 2019
@christopherthielen christopherthielen deleted the validation-message-jsx branch June 4, 2019 22:03
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Jun 5, 2019
8398d77 refactor(*): make accountExtractor return an array of strings (spinnaker#7068)
e6d1586 feat(core/presentation): support JSX for validationMessage in FormFields (spinnaker#7083)
78a9f72 fix(core): Disable rewriteLinks to allow proper event handling (spinnaker#7064)
3f21c1f feat(executions): Adding redirect for execution without application (spinnaker#7076)
f89d97d feat(core/presentation): Add "success" type to ValidationMessage (spinnaker#7082)
4e89a39 refactor(core/presentation): Consolidate Checklist and ChecklistInput components (spinnaker#7077)
9bbc002 feat(core/presentation): Support inline radio buttons (spinnaker#7078)
b142789 fix(triggers): Poll on execution id instead of event id (spinnaker#7067)
6af93e3 refactor(pipeline): Pipeline stage execution details to react (spinnaker#7063)
christopherthielen added a commit that referenced this pull request Jun 5, 2019
8398d77 refactor(*): make accountExtractor return an array of strings (#7068)
e6d1586 feat(core/presentation): support JSX for validationMessage in FormFields (#7083)
78a9f72 fix(core): Disable rewriteLinks to allow proper event handling (#7064)
3f21c1f feat(executions): Adding redirect for execution without application (#7076)
f89d97d feat(core/presentation): Add "success" type to ValidationMessage (#7082)
4e89a39 refactor(core/presentation): Consolidate Checklist and ChecklistInput components (#7077)
9bbc002 feat(core/presentation): Support inline radio buttons (#7078)
b142789 fix(triggers): Poll on execution id instead of event id (#7067)
6af93e3 refactor(pipeline): Pipeline stage execution details to react (#7063)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants