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): filter falsy error messages from errors object on tasks #7135

Merged
merged 1 commit into from
Jun 21, 2019
Merged

fix(core): filter falsy error messages from errors object on tasks #7135

merged 1 commit into from
Jun 21, 2019

Conversation

anotherchrisberry
Copy link
Contributor

Worth digging into the backend, but we ran into an internal error that resulted in "errors": [null], so we should filter those out and surface the "error" field, if it's present.

@marchello2000
Copy link
Contributor

marchello2000 commented Jun 21, 2019

lgtm, but you know i don't speak JS :)
but, i shall look at the orca sideof things

@anotherchrisberry
Copy link
Contributor Author

lgtm, but you know i don't speak JS :)

Yeah, that is why I tagged you on the PR, I just wanted an easy thumbs up, not a real review

@anotherchrisberry anotherchrisberry merged commit bbc1d06 into spinnaker:master Jun 21, 2019
@anotherchrisberry anotherchrisberry deleted the non-null-errors branch June 21, 2019 03:34
anotherchrisberry added a commit that referenced this pull request Jun 26, 2019
…94 (#7150)

* chore(core): Bump version to 0.0.377

d5425b4 refactor(core): virtualize execution rendering (#7140)
342087c feat(core): allow users to override pipeline graph positions (#7141)
fb58d70 fix(core): do not send a cloud provider on v2 search calls (#7142)
5d6d9aa chore(core): clarify clone stage help text (#7131)
745f0a1 fix(core): Display latest template in pipeline template list (#7145)
456172b fix(webhooks): addresses issue 3450 - introduce a delay before polling webhook (#7144)
7238d1d feat(core): Enable new artifacts workflow in bakeManifest (#7138)
45e5aa3 feat(artifacts): find multiple artifacts from single execution (#7139)
dae73da fix(core): provide key for repeating param JSX elements (#7136)
bbc1d06 fix(core): filter falsy error messages from errors object on tasks (#7135)
4782808 refactor(core): Reactify overrideTimeout (#7126)
0a3bd68 feat(core/presentation): Always call onBlur in Checklist to "mark as touched" (#7134)
cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
fa515dc fix(core): do not stretch provider logos in selection modal (#7128)

* chore(docker): Bump version to 0.0.42

8358010 fix(docker): Allow auto-switch to manual entry when refreshing images (#7120)

* chore(amazon): Bump version to 0.0.194

cdd6f23 chore(package): Just Update Prettier™
7e464a9 fix(amazon): Support SpEL in advanced capacity (#7124)
08e9506 chore(deck): Update to Typescript 3.4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants