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

fix(core/presentation): fix null reference in ReactSelectInput #7515

Merged

Conversation

christopherthielen
Copy link
Contributor

No description provided.

Copy link
Contributor

@ethanfrogers ethanfrogers left a comment

Choose a reason for hiding this comment

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

THANK YOU!

@christopherthielen christopherthielen merged commit c18f307 into spinnaker:master Oct 11, 2019
@christopherthielen christopherthielen deleted the react-select-input-NPE branch October 11, 2019 17:53
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Oct 11, 2019
c18f307 fix(core/presentation): fix null reference in ReactSelectInput (spinnaker#7515)
075bce1 feat(core/presentation): allow markdown in ValidationMessage
aef69cf fix(core/presentation): Make all DOMPurify'd links open in a new window (spinnaker#7511)
a082909 fix(bakeManifest/helm): rawOverrides option (spinnaker#7514)
752b1ab fix(deck): Show fail fast status code only if they are not pre-configured (spinnaker#7512)
d03ce48 fix(core/pipeline): fix Artifactory and Nexus trigger NPE These were throwing upon initial mount because 'error' wasn't defined.
a9469e9 refactor: Migrate user code to new validation message API
4c5f532 refactor(core/presentation): Remove validationStatus from forms apis
eafb077 refactor(core/presentation): Split forms interfaces into three files: - /forms/fields/interface.ts - /forms/inputs/interface.ts - /forms/layouts/interface.ts
2d6c9e7 feat(core/utils): extract firstDefined utility function
b553465 refactor(core/validation): Move ValidationMessage from core/validation to core/presentation/forms/validation
8518286 feat(core/presentation): Extract useValidationData hook
aad69b0 fix(core/presentation): Handle empty message in validation message functions, e.g., `errorMessage(undefined)`
8529faf fix(core): fix jumping cursor in trigger inputs (spinnaker#7486)
36192ab fix(core/pipeline): Pass Trigger validateFn to the trigger's Formik
christopherthielen added a commit that referenced this pull request Oct 11, 2019
c18f307 fix(core/presentation): fix null reference in ReactSelectInput (#7515)
075bce1 feat(core/presentation): allow markdown in ValidationMessage
aef69cf fix(core/presentation): Make all DOMPurify'd links open in a new window (#7511)
a082909 fix(bakeManifest/helm): rawOverrides option (#7514)
752b1ab fix(deck): Show fail fast status code only if they are not pre-configured (#7512)
d03ce48 fix(core/pipeline): fix Artifactory and Nexus trigger NPE These were throwing upon initial mount because 'error' wasn't defined.
a9469e9 refactor: Migrate user code to new validation message API
4c5f532 refactor(core/presentation): Remove validationStatus from forms apis
eafb077 refactor(core/presentation): Split forms interfaces into three files: - /forms/fields/interface.ts - /forms/inputs/interface.ts - /forms/layouts/interface.ts
2d6c9e7 feat(core/utils): extract firstDefined utility function
b553465 refactor(core/validation): Move ValidationMessage from core/validation to core/presentation/forms/validation
8518286 feat(core/presentation): Extract useValidationData hook
aad69b0 fix(core/presentation): Handle empty message in validation message functions, e.g., `errorMessage(undefined)`
8529faf fix(core): fix jumping cursor in trigger inputs (#7486)
36192ab fix(core/pipeline): Pass Trigger validateFn to the trigger's Formik
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
c18f307 fix(core/presentation): fix null reference in ReactSelectInput (spinnaker#7515)
075bce1 feat(core/presentation): allow markdown in ValidationMessage
aef69cf fix(core/presentation): Make all DOMPurify'd links open in a new window (spinnaker#7511)
a082909 fix(bakeManifest/helm): rawOverrides option (spinnaker#7514)
752b1ab fix(deck): Show fail fast status code only if they are not pre-configured (spinnaker#7512)
d03ce48 fix(core/pipeline): fix Artifactory and Nexus trigger NPE These were throwing upon initial mount because 'error' wasn't defined.
a9469e9 refactor: Migrate user code to new validation message API
4c5f532 refactor(core/presentation): Remove validationStatus from forms apis
eafb077 refactor(core/presentation): Split forms interfaces into three files: - /forms/fields/interface.ts - /forms/inputs/interface.ts - /forms/layouts/interface.ts
2d6c9e7 feat(core/utils): extract firstDefined utility function
b553465 refactor(core/validation): Move ValidationMessage from core/validation to core/presentation/forms/validation
8518286 feat(core/presentation): Extract useValidationData hook
aad69b0 fix(core/presentation): Handle empty message in validation message functions, e.g., `errorMessage(undefined)`
8529faf fix(core): fix jumping cursor in trigger inputs (spinnaker#7486)
36192ab fix(core/pipeline): Pass Trigger validateFn to the trigger's Formik
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants