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

Non-existent (Ghost) users Pull Requests crash Github tab #2014

Closed
kierandelaney opened this issue Mar 13, 2019 · 1 comment · Fixed by #2100
Closed

Non-existent (Ghost) users Pull Requests crash Github tab #2014

kierandelaney opened this issue Mar 13, 2019 · 1 comment · Fixed by #2100
Assignees
Labels

Comments

@kierandelaney
Copy link

kierandelaney commented Mar 13, 2019

Edit by @rsese to add alternate repro steps with a public repository where I found a now 👻-authored PR: atom/command-palette#76

  1. Clone https://github.com/atom/command-palette
  2. Toggle the GitHub tab GitHub: Toggle GitHub Tab (at this point Bye bye coffee lint and use standard command-palette#76 is filtered out and not shown in the Open pull requests list)
  3. GitHub: Open Issue or Pull Request and enter https://github.com/atom/command-palette/pull/76

The Git and GitHub tabs go blank and I see this error in console:

TypeError: Cannot read property 'url' of null
    at BarePullRequestDetailView.render (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4094739)
    at Qg (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2426345)
    at Og (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2426140)
    at Tg (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2429973)
    at bi (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2453979)
    at ci (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2454363)
    at Di (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2461253)
    at Yh (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2460633)
    at Xh (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2459654)
    at qf (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2458525)
    at Object.enqueueSetState (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2408169)
    at ReactRelayQueryRenderer.E.setState (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2336882)
    at n.handleDataChange (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4329082)
    at /Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4514840
    at Array.forEach (<anonymous>)
    at ReactRelayQueryFetcher._onQueryDataAvailable (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4514813)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4512609)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4547953)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Executor._next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4727251)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4725336)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at /Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4551491
    at <anonymous>

At this point, you need to reload the window to use the Git and GitHub tabs again.

1.36.0-beta-2 on macOS 10.12.6.


Prerequisites

Description

When a user deletes their account, any open pull requests they created will crash the github tab

Steps to Reproduce

  1. Create and add a user to a private repository
  2. Have the user open a pull request
  3. Delete the user account
  4. Open github tab

Expected behavior:

Expected the tab to show the pull requests and associate them with 'ghost' the generic Github user that is displayed on the PR within Github's UI.

Actual behavior:

Tab crashes and takes git tab with it

Reproduces how often:

Anytime you open the tab and the pull request remains open

Versions

All

Additional Information

We noticed this in a private repository, but I assume the issue will exist for public repositories as well

@rsese
Copy link
Contributor

rsese commented Apr 2, 2019

Thanks for the report @kierandelaney! I can reproduce with this public now ghost-authored pull request:

atom/command-palette#76

Repro steps:

  1. Clone https://github.com/atom/command-palette
  2. Toggle the GitHub tab GitHub: Toggle GitHub Tab (at this point Bye bye coffee lint and use standard command-palette#76 is filtered out and not shown in the Open pull requests list)
  3. GitHub: Open Issue or Pull Request and enter https://github.com/atom/command-palette/pull/76

The Git and GitHub tabs go blank and I see this error in console:

TypeError: Cannot read property 'url' of null
    at BarePullRequestDetailView.render (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4094739)
    at Qg (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2426345)
    at Og (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2426140)
    at Tg (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2429973)
    at bi (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2453979)
    at ci (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2454363)
    at Di (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2461253)
    at Yh (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2460633)
    at Xh (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2459654)
    at qf (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2458525)
    at Object.enqueueSetState (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2408169)
    at ReactRelayQueryRenderer.E.setState (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:2336882)
    at n.handleDataChange (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4329082)
    at /Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4514840
    at Array.forEach (<anonymous>)
    at ReactRelayQueryFetcher._onQueryDataAvailable (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4514813)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4512609)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4547953)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at Executor._next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4727251)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4725336)
    at Object.next (/Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4549167)
    at /Applications/Atom Beta.app/Contents/Resources/app/static/<embedded>:14:4551491
    at <anonymous>

At this point, you need to reload the window to use the Git and GitHub tabs again.

1.36.0-beta-2 on macOS 10.12.6.

@rsese rsese added the bug label Apr 2, 2019
@kuychaco kuychaco self-assigned this Apr 24, 2019
kuychaco added a commit that referenced this issue Apr 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants