damianavila / RISE

RISE: "Live" Reveal.js Jupyter/IPython Slideshow Extension
Other
3.7k stars 416 forks source link

decktape rise doesn't work #374

Open hrsma2i opened 6 years ago

hrsma2i commented 6 years ago

I read the official document and I execute the below command.

phantomjs decktape.js rise http://localhost:8888/notebooks/your/notebook.ipynb?token=5413981230123YourIndividualJupyterNotebookSessionToken412417923   /path/to/outputfile.pdf

But the below error occurred.

SyntaxError: Unexpected token 'const'

  phantomjs://code/decktape.js:5

How should I do?

Thanks.

damianavila commented 6 years ago

@astefanutti do you have any thoughts on this one?

astefanutti commented 6 years ago

@damianavila the RISE documentation still refers to DeckTape 1.x, while it instructs to clone the master branch. The documentation should be amended to either fix the clone step to refer to 1.x branch or to update the install instructions with DeckTape 2.x.

damianavila commented 6 years ago

Thanks, @astefanutti for your feedback. We will update our docs.

astefanutti commented 6 years ago

Great. Let me know if you need any help / review.

damianavila commented 6 years ago

Left some comments here: https://github.com/astefanutti/decktape/issues/110#issuecomment-414847438