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

reloading breaks easyform #6442

Closed
scottdraves opened this issue Dec 1, 2017 · 1 comment
Closed

reloading breaks easyform #6442

scottdraves opened this issue Dec 1, 2017 · 1 comment

Comments

@scottdraves
Copy link
Contributor

  1. open doc/groovy/EasyFormDemos.ipynb and run the 1st cell
  2. reload the page
  3. run the 1st cell again

this error appears on the JS console, and the "run" button fails:

Could not open comm  --  Error: Class beaker.tag.run not found in registry 
    at utils.js:880
    at new Promise (<anonymous>)
    at Object.load_class (utils.js:865)
    at CommManager.comm_open (comm.js:84)
    at i (jquery.min.js:4)
    at Kernel._handle_iopub_message (kernel.js:1223)
    at Kernel._finish_ws_message (kernel.js:1015)
    at kernel.js:1006
    at <anonymous>
(anonymous) @ utils.js:916
Promise rejected (async)
CommManager.comm_open @ comm.js:85
i @ jquery.min.js:4
Kernel._handle_iopub_message @ kernel.js:1223
Kernel._finish_ws_message @ kernel.js:1015
(anonymous) @ kernel.js:1006
Promise resolved (async)
Kernel._handle_ws_message @ kernel.js:1006
i @ jquery.min.js:4
kernel.js:1007 Couldn't process kernel message WrappedError {stack: "Error: Could not open comm↵    at new WrappedError…9561a88cca9375b7d86bf:20123:33↵    at <anonymous>", message: "Could not open comm", error_stack: Array(2)}
@scottdraves scottdraves added the Bug label Dec 1, 2017
@scottdraves scottdraves assigned lmitusinski and unassigned michalgce Dec 15, 2017
MariuszJurowicz pushed a commit that referenced this issue Dec 19, 2017
scottdraves pushed a commit that referenced this issue Dec 19, 2017
* #6442 fix comm target registration on page reload

* #6442 register msg handlers for existing comms

* #6442 remove shared constants
@scottdraves
Copy link
Contributor Author

classic notebook fixed, leaving open for lab fix.

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

No branches or pull requests

4 participants