Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

A discord bot #3

Merged
merged 3 commits into from
Feb 22, 2018
Merged

A discord bot #3

merged 3 commits into from
Feb 22, 2018

Conversation

Aaronstar-McClure
Copy link
Contributor

converted https://github.com/ATLauncher/discord-bot to work on pterodactyl. while i did not directly use his docker file, the install script was somewhat based off of it. docker github at https://github.com/Aaronstar-McClure/npm, auto built at https://hub.docker.com/r/aaronstar/npm/

@Aaronstar-McClure
Copy link
Contributor Author

working on an alert for incomplete configs now...

@parkervcp
Copy link
Collaborator

Would you mind horribly updating the description for the bot to have links to the ATL repos?

Also do you have links to the dockerfile for this? I am sure there is a way we can add a node container to the pterodactyl repo.

@parkervcp
Copy link
Collaborator

I have been working on tweaking this service myself while I wait.
So far I have

  1. Bot installing and running using node:8.9 alpine.
  2. Have a container for nodejs that I will most likely be doing a PR for to the main repo.
    The files are on my https://github.com/parkervcp/Containers repo.
  3. Added variables for the config.

@Aaronstar-McClure
Copy link
Contributor Author

alright, here are the links, origional bot github: https://github.com/ATLauncher/discord-bot origional bot dockerhub: https://hub.docker.com/r/atlauncher/discord-bot/ my version github: https://github.com/Aaronstar-McClure/npm my version dockerhub: https://hub.docker.com/r/aaronstar/npm/

@parkervcp
Copy link
Collaborator

@Aaronstar-McClure I am about to accept this PR but I will be tearing it apart and updating it with what I have.

Please take a look after I get my stuff up.

@Aaronstar-McClure
Copy link
Contributor Author

i used node 8, because that was what he used, im not sure which distro that was built off of, so all the commands were trial and error. i shyed away from config parseing because i had some trouble with it for a different egg. things of note though: the file in /configs/ needs to be renamed to remove the .example...

@Aaronstar-McClure
Copy link
Contributor Author

also, the install script uses node 8 because it needs to build the server before passing it to the run docker

@parkervcp parkervcp merged commit ab55503 into pelican-eggs:master Feb 22, 2018
parkervcp pushed a commit that referenced this pull request Dec 26, 2020
trenutoo pushed a commit that referenced this pull request May 31, 2021
Added new variables, fixed the current Libc++
@kkoshelev kkoshelev mentioned this pull request Dec 9, 2022
3 tasks
@szentigrad3 szentigrad3 mentioned this pull request Aug 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants