-
Notifications
You must be signed in to change notification settings - Fork 308
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
ExtensionApp: change extension_name to name. #232
Conversation
Thx @Zsailer. So for now this PR doesn't pick the python extension standard mecanism but we stick with the I have pulled the changes and updated the example. I fail with
|
That’s right, @echarles. I think the entry point discussion can be separated from this for now. We’ll discuss that at our next server meeting. I’ll update the examples directory next |
Cool! Having a way to create e.g. jupyter_lab_config,.py will be an excellent first step (PS: I won't be able to join today meeting) |
I've updated the examples (sorry for the delay, here). This should be ready for merge. |
Thx Zach. There are still some BTW As discussed, I will open a PR to run the examples by GitHub Actions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine for me. Thx Zach for doing this.
Thanks, @echarles. Merging! |
@Zsailer I have updated the jlab branches to latest master including this PR. It works, but when I change
I have nailed down the issue but we need to discuss about that. |
@Zsailer for example, if you change in the simple_ext1 the name and you say |
* when websocket closes, reopen a new one * stabilize kernel blocked websocket too * handle websockets closed on purpose * add new telemetry listener to consolidate some logic * create separate websockets for each telemetry event * turn telemetry listener into singleton and single websocket * add a test for telemetry listerner * add test for reconnecting when a window is not hidden * skip test that came from cookiecutter
ExtensionApp: change extension_name to name.
Changes the
extension_name
trait toname
in ExtensionApp.This addresses issues in #222 and #224. @echarles