Handle Error for issue note in the vulnerability modal
The following discussion from !110444 (merged) should be addressed:
-
@dftian started a discussion: (+1 comment) Since
issueLinks
defaults to[]
, the null-safe operator here is unnecessary, but more broadly, expanding on this comment, we needissueLinks
and if it's not provided, this is an error condition that we shouldn't hide. Thus, I think it'd be better if we wrote the code assuming thatissueLinks
exists, and if it doesn't we let the error fall through to the console:thanks for the patch! Yes, great call on this! If there are no
nodes
, like you said, it's a graphQL error and we should let that fall through. Perhaps we can do that more gracefully, maybe have an alert message. But we can handle that in a follow up, let me create that💪