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

Cancelling a push by clicking away from the dialog incorrectly asks for git credentials #725

Closed
ianhi opened this issue Aug 10, 2020 · 3 comments · Fixed by #777
Closed

Comments

@ianhi
Copy link
Collaborator

ianhi commented Aug 10, 2020

Description

In the below gif I interact with the push dialog in three different ways

  1. Let it finish
    • Works - no issues with credentials
  2. Dismiss using the dismiss button
    • Works as expected
  3. Cancel the full screen modal by clicking somewhere other than the push dialog
    • This incorrectly brings up the credentials dialog

pushing

Reproduce

With setting "blockWhileCommandExecutes": true:

  1. Enter a repo you have approriate credentials for
  2. click push button
  3. click somewhere on the screen
  4. Git credentials box pops up

Expected behavior

Credentials box does not pop up

Context

  • Python package version: 0.21.0a0
  • Extension version: 0.21.0a0
  • Git version: 2.25.1
  • Operating System and its version: PopOs 20.04

No interesting output in either console

@fcollonval
Copy link
Member

@ianhi Thank for testing the extension. I'm working on this one as part of #721

@ianhi
Copy link
Collaborator Author

ianhi commented Aug 10, 2020

If I just click ok on the credentials dialog then the push will complete:
pushing-2

but clicking multiple times will continue to spawn the box:
pushing-3

@ianhi ianhi changed the title Cancelling a push using the new modal incorrectly asks for git credentials Cancelling a push by clicking away from the dialog incorrectly asks for git credentials Aug 11, 2020
@ianhi
Copy link
Collaborator Author

ianhi commented Aug 11, 2020

From #721 (comment) this is not an issue with the new modal. It is partially and issue with jupyterlab dialogs and partially with how the push dialog handles cancelling of the dialog.

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

Successfully merging a pull request may close this issue.

2 participants