d-zone-org / d-zone

An ambient life simulation driven by user activity within a Discord server
ISC License
577 stars 74 forks source link

D-Zone & Heroku #55

Closed HottonJoris closed 5 years ago

HottonJoris commented 6 years ago

Hi,

First, thank you for your awesome work !

Second, is it possible to host d-zone server on Heroku ? I tried but it seems that client can't connect to servers websockets.

vegeta897 commented 6 years ago

Thank you for the compliment, I'm super happy that people enjoy this project.

I have never seen a working Heroku deployment of server-side D-Zone. I don't have personal experience with it so I can't offer a detailed explanation. Sorry!

I'll leave this open in case anyone who has managed to get it to work wants to share their advice.

HottonJoris commented 6 years ago

I try to merge UI & Server part (universal app) to avoid external websocket issue with Heroku. Hope it'll suceed !

Macleykun commented 6 years ago

Can you try going to the webadress of the app, but use http. Rather then https? I actully waited trying this myself due to the https thingie.

HottonJoris commented 6 years ago

Already try, the app only work in http (https = "insecure operation" error), i'm stuck on 'Connecting..' and got a timeout error for the ws connection

HottonJoris commented 6 years ago

I find a way to make it work with Heroku:

1) Create an express server into index.js and serve index.html file 2) Replace Websock with Socket.IO

Now the app is Universal !

Once it will be finished, i'll make it available for everyone, maybe on a new branch or another repo, like a fork.. IDK ? @vegeta897

Macleykun commented 6 years ago

Wow you did that fast!

Just a little idea: due to Heroku restarting every 24 hours (give or take). We can do the following smart trick:

a thought Run the .sh file to update the server list, on (re)boot. Then users don't have to update it locally, and then re-push the bot.

HottonJoris commented 6 years ago

You can find a working version here : https://world-of-nep.herokuapp.com/?s=default (Discord server is one of mine (FR) )

I didn't implemented "Presence" ATM, working on it. Now I didn't need to set IP & PORT for socket, everything on the same host (Back & UI), NodeJS universal app

edit: Now it works with https ! :)

Macleykun commented 6 years ago

Damn that looks perfect!

I also found out that you can set a custom domain too (world-of-nep.herokuapp.com could be world-of-nep.com)

This is also possible for a free tier it seems. https://devcenter.heroku.com/articles/custom-domains https://stackoverflow.com/questions/7170664/how-to-configure-heroku-application-dns-to-godaddy-domain

Only now left is to set a .com/dzone/ (that's what i prefer, but that's ma own issue :P)

HottonJoris commented 6 years ago

With express it's possible, you define that you get the index.html only when user is on /dzone for example

I need to discuss with @vegeta897 to determine if I make a new branch or a fork. You will see how it works

vegeta897 commented 6 years ago

This is very impressive, great work.

Your working link is stuck on connecting for me though.

I'd be happy to have it as a branch, but a fork would be OK too. I'm not sure which is more appropriate since I wouldn't be maintaining it as someone who doesn't use Heroku.

I would definitely like to have a tutorial page on the wiki for how to set it up too, if you're interested in writing that.

HottonJoris commented 6 years ago

Sometimes I need to restart Heroku server, IDK why (people got stuck on connecting like you)

Why not make a tutorial, I edited all the websockets part of your code (between main.js and index.js)

ATM i'm trying to make a Theme selector, because I really want to get boat actors and sea tiles ! (thx sea of thieves!)

I'll make a fork repo with my code and write a tutorial if people want to 'migrate' to the 'heroku' version

vegeta897 commented 6 years ago

Cool, looking forward to seeing your work!

HottonJoris commented 6 years ago

Server available https://world-of-nep.herokuapp.com/?s=default (it's working atm)

HottonJoris commented 6 years ago

I'd finish the Theme selector, you can try it (link above)

If you're stuck on 'Connecting..', refresh the page until it works (5 try max 🏷, I need to implement a timeout handler)

How to change theme :

Now, there is just one 'block' that change, I need to do the rest, but the system is implemented

@vegeta897 I will make the same for the 'normal' version (master branch) and make a pull request ? as u want

vegeta897 commented 6 years ago

Nice job, I want to see what that theme looks like when finished 😮

I want to handle theming a different way, and I will save it for the rewrite, but thank you for the offer.

Do you know what's causing the connection issues?

HottonJoris commented 6 years ago

IDK, I think it's the websocket connection which failed to reach the server for some reason. I need to investigate a bit on this subject

HottonJoris commented 6 years ago

image

Small update on Pirate theme, now I need to make details

Macleykun commented 6 years ago

So about heroku, i wanna try to get it to work. Of course without the https I can’t verfiy if it tottaly works..

If MechaPoulpe can share the https version that would be great! If the bot works on it, the same steps apply to the main.

HottonJoris commented 6 years ago

I need to make a tutorial when I get some time.

Don't worry I'll share what I've done

Le lun. 30 avr. 2018 à 08:42, Macley-Kun notifications@github.com a écrit :

So about heroku, i wanna try to get it to work. Of course without the https I can’t verfiy if it tottaly works..

If MechaPoulpe can share the https version that would be great! If the bot works on it, the same steps apply to the main.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-385322668, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr0rR4PPTycryoA7y9Tg9LFfMsyGsks5ttrI_gaJpZM4TPjxF .

Macleykun commented 6 years ago

Allright, ill try to get it to work today, if not this week. Instructions are universal and I’ll post them here when I get them down ;)

Met vriendelijke groet,

Op 30 apr. 2018 om 09:50 heeft Hotton Joris notifications@github.com het volgende geschreven:

I need to make a tutorial when I get some time.

Don't worry I'll share what I've done

Le lun. 30 avr. 2018 à 08:42, Macley-Kun notifications@github.com a écrit :

So about heroku, i wanna try to get it to work. Of course without the https I can’t verfiy if it tottaly works..

If MechaPoulpe can share the https version that would be great! If the bot works on it, the same steps apply to the main.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-385322668, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr0rR4PPTycryoA7y9Tg9LFfMsyGsks5ttrI_gaJpZM4TPjxF .

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

HottonJoris commented 6 years ago

The solution for Heroku was to use express server to render the UI and replace the websocket library used by socket.io (some code rework is necessary)

Le lun. 30 avr. 2018 à 10:51, Macley-Kun notifications@github.com a écrit :

Allright, ill try to get it to work today, if not this week. Instructions are universal and I’ll post them here when I get them down ;)

Met vriendelijke groet,

Op 30 apr. 2018 om 09:50 heeft Hotton Joris notifications@github.com het volgende geschreven:

I need to make a tutorial when I get some time.

Don't worry I'll share what I've done

Le lun. 30 avr. 2018 à 08:42, Macley-Kun notifications@github.com a écrit :

So about heroku, i wanna try to get it to work. Of course without the https I can’t verfiy if it tottaly works..

If MechaPoulpe can share the https version that would be great! If the bot works on it, the same steps apply to the main.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://github.com/vegeta897/d-zone/issues/55#issuecomment-385322668 , or mute the thread < https://github.com/notifications/unsubscribe-auth/AKQvr0rR4PPTycryoA7y9Tg9LFfMsyGsks5ttrI_gaJpZM4TPjxF

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-385342723, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr89Rut-nd3qTmvFA2tHaulaUQ1-bks5tttCIgaJpZM4TPjxF .

Macleykun commented 6 years ago

Ah your right, should've re-read the whole issue 😛

favna commented 6 years ago

I just found this issue and it looks very interesting to host this off of Heroku. Will you still be sharing the code rework you did as well as setup instructions @MechaPoulpe ? I found your mirrored version of the code (not sure why not forked but hey you do you, it's ISC license after all) but while I would assume those are the base files Heroku is using it is not clear what is required for that to run for other people

HottonJoris commented 6 years ago

The code is host on github (for my personal use) you can use it as you want. There are some differences : -theme manager; -socket.io instead of websocket library; - universal code: server render the UI

HOTTON Joris - Découvrez mon CV en ligne http://www.doyoubuzz.com/joris-hotton

2018-05-14 15:27 GMT+02:00 Favna notifications@github.com:

I just found this issue and it looks very interesting to host this off of Heroku. Will you still be sharing the code rework you did as well as setup instructions @MechaPoulpe https://github.com/MechaPoulpe ? I found your mirrored version of the code (not sure why not forked but hey you do you, it's ISC license after all) but while I would assume those are the base files Heroku is using it is not clear what is required for that to run for other people

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-388815936, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr5L92RWfTCHYZlSPBe-VpT-PZZt4ks5tyYZHgaJpZM4TPjxF .

Macleykun commented 6 years ago

it is not clear what is required for that to run for other people

For Heroku we must need a procfile that tells Heroku how to run the app. I did some quick researching and found out that running a static website will be doable with some workarounds.

After we figured out how to run the app, we could improve the way of building/deploying by doing this part on Heroku's end. What i mean with that is that the commands: npm install --no-optional and npm run-script build should be run when push the build git push heroku master.

What i know so far, is that with the procfile we could say: bash npminstall.sh && npm start & index.html. But i want to see if we could do better. For now i can only offer small help due to my studies.

Macleykun commented 6 years ago

Also, If we plan to use a script, when we run the app. It will re-download everything from npm install/npm run-script build everytime we boot the app. (So less fast bootup times, guaranteed to work with my knowledge) If we plan to use it while we push the files to Heroku, it will only affect the build time (note that if there is nothing new with the commands, the time completing them will decrease! Yet i fail still how to exactly do this)

Might need to study this: https://devcenter.heroku.com/articles/nodejs-support#build-behavior

favna commented 6 years ago

Did some more figuring out myself as well and I managed to run it off of a github page with custom domain for the client and the server is running off of my VPS droplet at DigitalOcean (which already hosts my primary site anyway).

This solution will work just fine for me tbh. Would drop the link but I gotta do some config stuff first..

favna commented 6 years ago

Coming back to my comment from yesterday I did the aforementioned fixing so now I have ribbonzone-server which is the server and ribbonzone-client as client. The client is also a submodule on the server so they are all included when you'd recursively clone the server.

The production environment is

If any other readers are interested in how to config my version, you need

vegeta897 commented 6 years ago

Impressive work! Do you have any intention to do further customization or enhancements?

favna commented 6 years ago

Well yes there is one thing that caught my attention that I may improve, mobile support. It is not possible to press the [Server] or [?] buttons on mobile and for as far as I've been able to tell the text boxes for messages are also not rendered.

That said, I also have to set my own priorities and I have my Discord bot to tend to as well and I'm also trying to start something with Amazon Alexa skills since the only package that links Discord with Alexa right now is well... not updated. Not to mention work and social life ¯\_(ツ)_/¯

Macleykun commented 6 years ago

I think i have it working almost. It seems that this is the last bit holding me back from getting it to work dedicated:

2018-05-30T11:43:02.317602+00:00 app[web.1]: > d-zone@0.1.65 start /app/d-zone
2018-05-30T11:43:02.317604+00:00 app[web.1]: > node index.js
2018-05-30T11:43:02.317605+00:00 app[web.1]: 
2018-05-30T11:43:02.410481+00:00 app[web.1]: Optimizing defaults for 1X dyno...
2018-05-30T11:43:02.526384+00:00 app[web.1]: Initializing server
2018-05-30T11:43:02.574099+00:00 app[web.1]: 4 processes at 128MB memory limit.
2018-05-30T11:43:02.583418+00:00 app[web.1]: Starting php-fpm...
2018-05-30T11:43:02.870106+00:00 app[web.1]: 2018-05-30T11:43:02.868Z 'Logged in as: Delta - (341194927935586305)'
2018-05-30T11:43:02.872137+00:00 app[web.1]: Connected to 8 server(s)
2018-05-30T11:43:02.899642+00:00 app[web.1]: events.js:183
2018-05-30T11:43:02.899646+00:00 app[web.1]:       throw er; // Unhandled 'error' event
2018-05-30T11:43:02.899648+00:00 app[web.1]:       ^
2018-05-30T11:43:02.899649+00:00 app[web.1]: 
2018-05-30T11:43:02.899651+00:00 app[web.1]: Error: listen EACCES 0.0.0.0:443
2018-05-30T11:43:02.899655+00:00 app[web.1]:     at Object._errnoException (util.js:992:11)
2018-05-30T11:43:02.899657+00:00 app[web.1]:     at _exceptionWithHostPort (util.js:1014:20)
2018-05-30T11:43:02.899659+00:00 app[web.1]:     at Server.setupListenHandle [as _listen2] (net.js:1338:19)
2018-05-30T11:43:02.899660+00:00 app[web.1]:     at listenInCluster (net.js:1396:12)
2018-05-30T11:43:02.899662+00:00 app[web.1]:     at doListen (net.js:1505:7)
2018-05-30T11:43:02.899664+00:00 app[web.1]:     at _combinedTickCallback (internal/process/next_tick.js:141:11)
2018-05-30T11:43:02.899665+00:00 app[web.1]:     at process._tickCallback (internal/process/next_tick.js:180:9)
2018-05-30T11:43:02.909193+00:00 app[web.1]: npm ERR! code ELIFECYCLE
2018-05-30T11:43:02.909478+00:00 app[web.1]: npm ERR! errno 1
2018-05-30T11:43:02.910525+00:00 app[web.1]: npm ERR! d-zone@0.1.65 start: `node index.js`
2018-05-30T11:43:02.910627+00:00 app[web.1]: npm ERR! Exit status 1
2018-05-30T11:43:02.910792+00:00 app[web.1]: npm ERR! 
2018-05-30T11:43:02.910902+00:00 app[web.1]: npm ERR! Failed at the d-zone@0.1.65 start script.
2018-05-30T11:43:02.911006+00:00 app[web.1]: npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
2018-05-30T11:43:02.915049+00:00 app[web.1]: 
2018-05-30T11:43:02.915191+00:00 app[web.1]: npm ERR! A complete log of this run can be found in:
2018-05-30T11:43:02.915275+00:00 app[web.1]: npm ERR!     /app/.npm/_logs/2018-05-30T11_43_02_912Z-debug.log
2018-05-30T11:43:04.585609+00:00 app[web.1]: Starting httpd...

I think due to not having a static ip. It can't loopback itself (127.0.0.1:443). If we could use the hostname instead, it should work.

HottonJoris commented 6 years ago

Check my repo with my implementation of d-zone

https://github.com/MechaPoulpe/world-of-nep?files=1

Only one app (server & client unified) based on express server with socket.io

Working in heroku

Le mer. 30 mai 2018 à 13:48, Macley-Kun notifications@github.com a écrit :

I think i have it working almost. It seems that this is the last bit holding me back from getting it to work dedicated:

2018-05-30T11:43:02.317602+00:00 app[web.1]: > d-zone@0.1.65 start /app/d-zone 2018-05-30T11:43:02.317604+00:00 app[web.1]: > node index.js 2018-05-30T11:43:02.317605+00:00 app[web.1]: 2018-05-30T11:43:02.410481+00:00 app[web.1]: Optimizing defaults for 1X dyno... 2018-05-30T11:43:02.526384+00:00 app[web.1]: Initializing server 2018-05-30T11:43:02.574099+00:00 app[web.1]: 4 processes at 128MB memory limit. 2018-05-30T11:43:02.583418+00:00 app[web.1]: Starting php-fpm... 2018-05-30T11:43:02.870106+00:00 app[web.1]: 2018-05-30T11:43:02.868Z 'Logged in as: Delta - (341194927935586305)' 2018-05-30T11:43:02.872137+00:00 app[web.1]: Connected to 8 server(s) 2018-05-30T11:43:02.899642+00:00 app[web.1]: events.js:183 2018-05-30T11:43:02.899646+00:00 app[web.1]: throw er; // Unhandled 'error' event 2018-05-30T11:43:02.899648+00:00 app[web.1]: ^ 2018-05-30T11:43:02.899649+00:00 app[web.1]: 2018-05-30T11:43:02.899651+00:00 app[web.1]: Error: listen EACCES 0.0.0.0:443 2018-05-30T11:43:02.899655+00:00 app[web.1]: at Object._errnoException (util.js:992:11) 2018-05-30T11:43:02.899657+00:00 app[web.1]: at _exceptionWithHostPort (util.js:1014:20) 2018-05-30T11:43:02.899659+00:00 app[web.1]: at Server.setupListenHandle [as _listen2] (net.js:1338:19) 2018-05-30T11:43:02.899660+00:00 app[web.1]: at listenInCluster (net.js:1396:12) 2018-05-30T11:43:02.899662+00:00 app[web.1]: at doListen (net.js:1505:7) 2018-05-30T11:43:02.899664+00:00 app[web.1]: at _combinedTickCallback (internal/process/next_tick.js:141:11) 2018-05-30T11:43:02.899665+00:00 app[web.1]: at process._tickCallback (internal/process/next_tick.js:180:9) 2018-05-30T11:43:02.909193+00:00 app[web.1]: npm ERR! code ELIFECYCLE 2018-05-30T11:43:02.909478+00:00 app[web.1]: npm ERR! errno 1 2018-05-30T11:43:02.910525+00:00 app[web.1]: npm ERR! d-zone@0.1.65 start: node index.js 2018-05-30T11:43:02.910627+00:00 app[web.1]: npm ERR! Exit status 1 2018-05-30T11:43:02.910792+00:00 app[web.1]: npm ERR! 2018-05-30T11:43:02.910902+00:00 app[web.1]: npm ERR! Failed at the d-zone@0.1.65 start script. 2018-05-30T11:43:02.911006+00:00 app[web.1]: npm ERR! This is probably not a problem with npm. There is likely additional logging output above. 2018-05-30T11:43:02.915049+00:00 app[web.1]: 2018-05-30T11:43:02.915191+00:00 app[web.1]: npm ERR! A complete log of this run can be found in: 2018-05-30T11:43:02.915275+00:00 app[web.1]: npm ERR! /app/.npm/_logs/2018-05-30T11_43_02_912Z-debug.log 2018-05-30T11:43:04.585609+00:00 app[web.1]: Starting httpd...

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-393131992, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr--u4H4sHH-Veeu5YR1YB30uG7Woks5t3oclgaJpZM4TPjxF .

Macleykun commented 6 years ago

o, well, ig i use that then >.>

HottonJoris commented 6 years ago

Sry I don't have the time to write a tutorial for make it working on heroku. If someone want to write it based on my repo, go on 😍

Le mer. 30 mai 2018 à 14:29, Macley-Kun notifications@github.com a écrit :

o, well, ig i use that then >.>

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-393143175, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr57Ge4q-0-7fQ0mkuZMlycXhOPb_ks5t3pC5gaJpZM4TPjxF .

Macleykun commented 6 years ago

Well sure, but i still need to get the initial d-zone working. I'm quite sure it has to do with the ip not being a name. I think it would be this because i made the fredboat heroku instructions. The backend only could connect with the HOST NAME, and only trough a port like 80 or 433. I'm not sure how much harder it would be to use a hostname instead of IP for d-zone...

or.. do we need that? https://elements.heroku.com/addons/fixie

Macleykun commented 6 years ago

After alot of trial and error i didn't succeed to get it to work. Due to studies i'm unable to spent more time into it. Techguy12 🔥 will take over my findings and hopefully he finds a way to get it to work.

So far, the web part works. The main part doesn't Might be something around the port/ip/config part. It's surely the app part but where exactly i'm not sure.

HottonJoris commented 6 years ago

I try a lot of way to make it work, after 1 day I rework websocket implementation with socket.io (I prefer that library) and make the app universal (I'm lazy to manage 2 code 😂)

Le mer. 30 mai 2018 à 19:27, Macley-Kun notifications@github.com a écrit :

After alot of trial and error i didn't succeed to get it to work. Due to studies i'm unable to spent more time into it. Techguy12 🔥 will take over my findings and hopefully he finds a way to get it to work.

So far, the web part works. The main part doesn't Might be something around the port/ip/config part. It's surely the app part but where exactly i'm not sure.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-393248842, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvrydDkri9FxpNheYZpzeuhrTX5c3kks5t3tZ-gaJpZM4TPjxF .

Macleykun commented 6 years ago

logs.txt d-zone.zip instructions.txt For anyone else intrested, you can use these files to get the web part working on Heroku. I do recommand replacing the files in web with your own. logs show a working webservers. instructions tell you what commands to use to get your files into Heroku.

If anyone finds the solution to get the core of d-zone working on Heroku, then d-zone should be able to work on Heroku as a easy and free way for everyone to enjoy.

HottonJoris commented 6 years ago

As I'd told, replace Websocket library with socket.io and serve both with an Express server make it work well on Heroku

HOTTON Joris - Découvrez mon CV en ligne http://www.doyoubuzz.com/joris-hotton

2018-06-05 11:07 GMT+02:00 Macley-Kun notifications@github.com:

logs.txt https://github.com/vegeta897/d-zone/files/2071629/logs.txt d-zone.zip https://github.com/vegeta897/d-zone/files/2071630/d-zone.zip instructions.txt https://github.com/vegeta897/d-zone/files/2071631/instructions.txt For anyone else intrested, you can use these files to get the web part working on Heroku. I do recommand replacing the files in web with your own. logs show a working webservers. instructions tell you what commands to use to get your files into Heroku.

If anyone finds the solution to get the core of d-zone working on Heroku, then d-zone should be able to work on Heroku as a easy and free way for everyone to enjoy.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/vegeta897/d-zone/issues/55#issuecomment-394637686, or mute the thread https://github.com/notifications/unsubscribe-auth/AKQvr0gQcOYi4yiiuaRVCBt4AwXoUfrJks5t5kplgaJpZM4TPjxF .

Macleykun commented 6 years ago

Any steps how to do that? Is it just a copy paste of another library or..?

Not a developer here, so i'm a bit new to this ^^;;

Macleykun commented 6 years ago

Or do you just want me to get it working on your repo?

Normally i try to get the original working. But if vegeta uses the same websocket library like you do, and it all works i'm ok to do it on your repo then.

Macleykun commented 6 years ago

What kind of websocket does D-Zone use?

vegeta897 commented 6 years ago

Regular ws://, non secure.

Macleykun commented 6 years ago

I've tried everything, but i think it's not possible to host it fully, unless the following would happen:

The solution for Heroku was to use express server to render the UI and replace the websocket library used by socket.io (some code rework is necessary)

It's easy to host the web part in heroku, and put the core somewhere else. But using Heroku only for webhosting isn't really that good.

Hope that D-Zone can run on Heroku some day 👍

NNTin commented 6 years ago

I got a proof of work for hosting d-zone on Heroku. I will create a pull request later tonight or tomorrow. d-zone: https://nntindzone.herokuapp.com/?s=default heroku deploy link: https://heroku.com/deploy?template=https://github.com/nntin/d-zone/tree/heroku github: https://github.com/NNTin/d-zone/tree/heroku

What I did: Added to the websocket "server" a http server using express. It hosts the static website. You can serve ws/wss and http/https on Heroku natively, so you don't need any certificates. Port is handled by Heroku so the websocket link has no port. Also added a bunch of files back into git such as config, bundle.js etc. Else you would have to manually commit the config files to Heroku git which isn't beginner friendly.

Can you create a heroku branch from your master? I will do my pull request there. I might have removed some code you need (SSL certification) and I'm not happy with my solution in general. It still needs a bit of refactoring/re-organisation/addition before it is ready for main branch. Also the discord server id is hardcoded into the config. That should be customizable from env var in Heroku. The heroku button link can however be put into the ReadMe.

Macleykun commented 6 years ago

Truely amazing work! You’ve done it in less then a day! I’m super happy that you got it to work and can’t wait to move over to heroku with d-zone!

One tiny suggestion, I made and-zone logo that you guys can have: https://deltabot.me/about/images/D-zone.png Suggest putting this in the git itself, so it is in a safe place :-)

Again thank you so so much! I’m super happy with the result you’ve brought!

vegeta897 commented 6 years ago

Awesome work @NNTin! I didn't occur to me that you could have the http and ws on the same url. I'll create a branch. Thanks so much for your contribution!

Macleykun commented 5 years ago

I think this can be closed :P

vegeta897 commented 5 years ago

Okie doke