Skip to content

Handle missing nwos returned from graphql query#1366

Merged
aeisenberg merged 1 commit intomainfrom
aeisenberg/handle-missing-nwo
May 30, 2022
Merged

Handle missing nwos returned from graphql query#1366
aeisenberg merged 1 commit intomainfrom
aeisenberg/handle-missing-nwo

Conversation

@aeisenberg
Copy link
Copy Markdown
Contributor

@aeisenberg aeisenberg commented May 27, 2022

This should avoid an error hit by @jf205.

Checklist

  • [n/a] CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • [n/a] Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@aeisenberg aeisenberg requested a review from a team as a code owner May 27, 2022 20:14
robertbrignull
robertbrignull approved these changes May 30, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@robertbrignull robertbrignull left a comment

Choose a reason for hiding this comment

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

LGTM

@aeisenberg aeisenberg merged commit 0d9f4e8 into main May 30, 2022
@aeisenberg aeisenberg deleted the aeisenberg/handle-missing-nwo branch May 30, 2022 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@robertbrignull robertbrignull robertbrignull approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aeisenberg @robertbrignull