move from config.json to .env system, update to discordjs@14, add linux instructions#136
Open
kxtzownsu wants to merge 7 commits into
Open
move from config.json to .env system, update to discordjs@14, add linux instructions#136kxtzownsu wants to merge 7 commits into
kxtzownsu wants to merge 7 commits into
Conversation
DiscordJS @ v13 and UUID @ v9 is now deprecated. This commit updates us to DiscordJS @ v14 and UUID @ v11. (uuid@11 is the latest we can go, as past that CommonJS is deprecated). This commit also switches us to use bcryptjs, which doesn't require any API changes. Finally, this commit will switch us to use a proper `.env` configuration style while still making sure most of the code doesn't need to be changed to support this new style. Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
Previously, we were still using DJS@v13-specific APIs. Now, this uses the proper DJS@v14 APIs. Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
This allows us to change the XMPP port if needed. (e.g: routing via nginx) Signed-off-by: kxtzownsu <kxtzownsu@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
When I was attempting to run Reload Backend, I noticed there was no Linux support, prompting me to start a PR.
Whilst I was trying to run it on Linux, I noticed some of the packages were marked as deprecated, so I updated them.
I also noticed some issues with the configuration:
.gitignore, allowing anyone to accidentally commit their info (e.g: their bot token, game server IPs, etc, etc).envthat most projects already useI updated the config system to now use
.envrather than the old JSON system, while keeping code changes to as little as possible viaconfig.js.(P.S: I also made it where
.envis in.gitignore)