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

Draft command #519

Conversation

KrzysztofSikoraCodete
Copy link
Contributor

@KrzysztofSikoraCodete KrzysztofSikoraCodete commented Jul 27, 2020

Related with #460

This is a draft pull request.

Flow works, however solution is not complete.
Peek 2020-07-27 16-14

@jtpio
Copy link
Member

jtpio commented Jul 28, 2020

Thanks @KrzysztofSikoraCodete.

however solution is not complete

What do you think is missing?

@KrzysztofSikoraCodete
Copy link
Contributor Author

@jtpio isEnabled method has hardcooded flag as "true". In all other commands we use there threadHasStopped(). Maybe in current command is not important?

@jtpio
Copy link
Member

jtpio commented Sep 30, 2020

Closing as we should now make changes to https://github.com/jupyterlab/jupyterlab, in preparation for #536

@jtpio jtpio closed this Sep 30, 2020
@jtpio
Copy link
Member

jtpio commented Sep 30, 2020

Thanks @KrzysztofSikoraCodete for working on this!

@jtpio jtpio added this to the Reference milestone Sep 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants