Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Error when trying to view a pull request #1776

Closed

Description

@Chew

Prerequisites

I even like asked on slack and they said come here.

Description

I went to checkout a pull request in my repository, however when I clicked on it, it failed to load.

Steps to Reproduce

These probably won't work, it's a unique situation

  1. Clone an repository with one open PR and open the project in atom.
  2. Open GH tab
  3. Click a pull request?? Not sure.

Expected behavior:

It to view the pull request as normal.

Actual behavior:

Both the github tab and main pane turn black and do nothing until I restart Atom.

Reproduces how often:

Only on one specific pull request. It did another from the same repository perfectly fine.

Versions

Versions? check it out

apm  2.1.2
npm  6.2.0
node 8.9.3 x64
atom 1.32.1
python 3.6.4
git 2.18.0

Additional Information

Here's a fun error I got.

<embedded>:14 TypeError: Cannot read property 'owner' of null
    at BareIssueishDetailController.nextCheckoutOp (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:3679469)
    at BareIssueishDetailController.render (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:3678190)
    at Gg (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2438932)
    at Kg (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2443812)
    at th (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2453366)
    at uh (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2453673)
    at Rh (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2457250)
    at Uh (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2456721)
    at Sh (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2456547)
    at wh (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2455920)
    at kg (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2455167)
    at Object.enqueueSetState (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2430758)
    at ReactRelayQueryRenderer.F.setState (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2356627)
    at n.handleDataChange (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:3998669)
    at ReactRelayQueryFetcher._onQueryDataAvailable (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4149708)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4147379)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4167092)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4170895
    at _subscribe (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168783)
    at RelayObservable.subscribe (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4169034)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4169660)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4167092)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4170895
    at _subscribe (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168783)
    at RelayObservable.subscribe (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4169034)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4169660)
    at Object.next (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4168385)
    at /Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:4170810
    at <anonymous>

This error has occurred on my home mac and my work ubuntu laptop.

yes officer this specific pull request right here is the one causing me problems

Activity

  1. smashwilson commented on Nov 5, 2018

    @smashwilson
    Contributor

    Ahh looks like we're assuming that some GraphQL node is required when it's actually nullable (... or the API changed). Thanks for the report, and also for giving us the specific pull request, that's super helpful 🙇

  2. self-assigned this
    on Nov 7, 2018
  3. annthurium commented on Nov 7, 2018

    @annthurium

    thanks @Chewsterchew for reporting! Working on a fix.

    In the mean time, a workaround is to use the "open issue or pull request" command from the command palette, which for some reason works.

    screen shot 2018-11-06 at 5 06 40 pm

  4. annthurium commented on Dec 6, 2018

    @annthurium

    Just merged a fix, though it will take a bit for the fix to make its way from beta to stable.

    I'm going to close, but feel free to reopen if there are additional issues or concerns.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions