Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

DJS v14, slash commands & GitHub App integration #106

Merged
merged 120 commits into from
Mar 3, 2024
Merged

DJS v14, slash commands & GitHub App integration #106

merged 120 commits into from
Mar 3, 2024

Conversation

dsevillamartin
Copy link
Member

  • Upgrade from djs v12 to v14
  • Use slash commands for most things
    • A couple text-only commands remain, e.g. deploying slash commands & telling user to use the slash commands instead
  • Deprecate the regular endpoint for webhooks (changed from / to /hook/legacy -- can redirect internally with eg. nginx)
    • They should still work. Cannot register new orgs/repos with init anymore
  • Add two new endpoints for webhooks: /hook for GitHub App, /hook/channel/:ids for specifying destination through GitHub hook
  • Registering repos/orgs to channels now requires GitHub OAuth to verify access to said repo
  • Updated some Discord events: add meta, better release handling, better vulnerability handling, remove issue/pr description from updates...

@dsevillamartin
Start working on updating for djs v13
e035e44 
Add intents, partials, collection updates, change .send(embed) to { embeds: [embed] }...
@dsevillamartin
wip
659ea2f
@dsevillamartin
more wip, properly register commands & owner command perms, make ping…
a014c96 
… cmd work
@dsevillamartin
discord: commands > update reload
167d42a
@dsevillamartin
discord: commands > update stats
9b28bac
@dsevillamartin
discord: modules > update UnhandledError
0d51744
@dsevillamartin
discord: commands > change reload - use choices for option
31f165f
@dsevillamartin
discord: commands > update clean
269c254
@dsevillamartin
discord: commands > change reload - fix some error handling
6153db5
@dsevillamartin
discord: commands > update init
0dbc348
@dsevillamartin
discord: commands > update remove
f8d369c
@dsevillamartin
discord: modules > fix RunCommand guild logic
ac26b7b
@dsevillamartin
discord: commands > update initorg
ca8366e
@dsevillamartin
discord: commands > update removeorg
ddd2e30
@dsevillamartin
discord: commands > fix initorg
16a0bb2
@dsevillamartin
discord: commands > update github issue
7a3d65c
@dsevillamartin
discord: commands > update github pr
0cb9a9d
@dsevillamartin
discord: commands > update search
c229063
@dsevillamartin
discord: commands > remove help
3189451
@dsevillamartin
discord: commands > update invite
be3b89d
@dsevillamartin
discord: commands > update org
c1657fd
@dsevillamartin
discord: commands > update update
2dc0d55
@dsevillamartin
discord: update command registering code & allow for few message-only…
2d997ed 
… commands for owner
@dsevillamartin
discord: commands > update conf
151c003
@dsevillamartin
discord: commands > create deploy
767c9f6
@dsevillamartin
discord: commands > mark repo actions as guild only
e1b3c11
@dsevillamartin
discord: commands > revert update back to non-slash command
b06f959
@dsevillamartin
discord: add DISCORD_OWNER_ID env
d3aa452
@dsevillamartin
discord: remove debug log
0e10008
@dsevillamartin
add WEB_HOST env for easy domain switching
69138a9
dsevillamartin and others added 28 commits January 1, 2024 18:15
@dsevillamartin
add node engine to package.json
476c2bc
@dsevillamartin
github: events > handle release drafts & prereleases (#66)
72d7b89
@dsevillamartin
remove 'mongoose' dep
9003de7
@dsevillamartin
web > escape GitHub webhook input when sending response back
bdcbdbf
@dsevillamartin
discord > allow extending perms to other users with slash command perms
c06e9bd
@dsevillamartin
web: setup > add ratelimiting to routes
9f5b53c
@dsevillamartin
web > add 'noopener noreferrer' to external links
ff4a409
@dsevillamartin
update README to explain gh app perms
3f65000
@dsevillamartin
web : setup > show warning if connection no longer accessible through…
d14219a 
… github app
@dsevillamartin
db > add some checks to Channel methods to prevent errors
9fb3dc2
@dsevillamartin
add log to beginning of program & fix error when invalid discord token
c4e58f5
@dsevillamartin
github: events > remove description from issues and pr edit events
4b5ab6b
@dsevillamartin
web > fix github ip check when no token is provided; autofail now
c81526a
@dsevillamartin
github: events > add events to handle reviews, review threads & comments
abbcb6b
@dsevillamartin
run prettier & return 500 if event failed to process
2f8e2cc
@dsevillamartin
web > add 'purge' functionality to allow disconnecting install from c…
2b9a90d 
…hannels
@dsevillamartin
db: migrations > fix ignore_unknown default causing foreign key issue
8d55f3a 
See more at knex/knex issue 5367
@dsevillamartin
web > fix hook code for checking secrets & channel endpoint
6da1456
@dsevillamartin
github: events > update vulnerability alerts & add more specific urls…
d90abf3 
… to some events
@dsevillamartin
update README with more details about GitHub App permissions
15f945c
@dsevillamartin
discord: commands > add non-slash 'help' command
5bb13c2
@dsevillamartin
web > fix next() not being defined for hook error handling
ba5adf0
@dsevillamartin
discord: commands > add more aliases to text-only 'help'
e0dc510
@dsevillamartin
web > don't verify channel existence & type in GET /hook/channel/:ids
18f975a
@dsevillamartin
db: migrations > fix missing data type for ignore_unknown column
ebd61cc
@dsevillamartin
db: migrations > remove discord 'name' column from guilds & channels
f36b6ef
@dsevillamartin
db: migrations > remove guild 'prefix' column
aa5a17a
@dsevillamartin
Merge branch 'rewrite' into djs1333ithub-app
fbda24a
@dsevillamartin dsevillamartin merged commit b9eebd9 into rewrite Mar 3, 2024
1 check passed
@dsevillamartin dsevillamartin mentioned this pull request Mar 3, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dsevillamartin