Some simple changes #261
Some simple changes #261
Conversation
Update docker files, upgrade to djs v14, and node v18
TheAlienKnight
requested review from
carolinaisslaying,
IceeMC and
advaith1
as code owners
|
You are correct, it seems my docker build had not updated (I guess I forgot to prune it) and it does in fact, not operate on v14 as expected. I'll make the corrected changes tonight! Though it would sure help if discord.js documented the new discord.js/rest system, since it's entirely empty :^) |
Actually change things to work with discord.js v14
|
Added correct changes for discord.js v14! All of these have actually been tested correctly this time (lol) - my bad about earlier. The changes should be broken down easily with the commit messages, and side-by-side comparisons! |
Use proper enums
|
Is this on pause or is it safe to merge this and work on the remaining patches at a later date? Also, what else needs to be done? Would like to work on some stuff myself if possible so I'm at least somewhat productive in my spare time 😅 |
|
Last I checked these fixes worked, though I had a server raid card go bad not too long ago and it screwed up the VM I had setup for testing this (didn't have a backup for it), so I can't 100% verify it works with the latest API changes. From what I can see though it should work fine! |
| @@ -94,7 +94,6 @@ | |||
|
|
|||
| const input = document.querySelector("input.input.is-rounded.is-dark#searchInput"); | |||
|
|
|||
| if (input.value.match(/gay(\s+)?(men\s+)?.*/i)) return window.location.href = "/users/302604426781261824"; | |||
There was a problem hiding this comment.
lmfao I forgot this code still existed
Some simple changes, mostly in prep for some other fixes/additions discussed in the community discord
These changes have been tested and work properly on Ubuntu 22.04.2 LTS