yusukebe / revealgo

Markdown driven presentation tool written in Go!
MIT License
237 stars 22 forks source link

Support slide multiplex #8

Closed jossemarGT closed 3 years ago

jossemarGT commented 3 years ago

Background

I have used reveal.js for presentations, and I always have wished to not rely on having a big screen or any sort of projector to do so. The multiplex feature gives the presenter a little more flexibility by letting the audience follow the presentation on their own devices.

Description

I am adding all the required configurations and dependencies to enable reveal.js multiplex plugin.

When the --multiplex parameter is provided, revealgo adds the /master/ endpoint that renders exactly the same slides. Still, these ones can push their state to the server while others under / only follow them.

Heads up! This requires an old version of socket.io (v1.6.0) since that is the latest one that github.com/googollee/go-socket.io supports :man_shrugging: .

NOTE: This is a draft PR since I have a question for you in the commets section.

jossemarGT commented 3 years ago

Hi, it's me again. I am aware you might be busy, but I realize I might not have checked if you are interested on this feature on the first place. If that is the case, feel free to say "no thanks", I can keep my fork without any problem.

Stay safe o/

yusukebe commented 3 years ago

Sorry for reply is so late. I tried using revealgo with multiplex feature by your PR's. Feels very good. I'll merge it!

jossemarGT commented 3 years ago

Hey @yusukebe I did not mean to rush you, I am sorry for that. But still, I thank you for accepting my contribution :octocat: