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

Turn bots into modules and streamline the development process #140

Closed
wants to merge 83 commits into from
Closed

Turn bots into modules and streamline the development process #140

wants to merge 83 commits into from

Conversation

iminlikewithyou
Copy link
Member

@iminlikewithyou iminlikewithyou commented Jan 12, 2024 •
edited
Loading

just to clarify. the branch is called sleuth because it was originally created to make moderation commands, but it became necessary to make these features first, so the name is no longer completely accurate to what the branch is aiming to do

@iminlikewithyou
Add Sleuth to tsconfig
2bfd742
@iminlikewithyou
Create Sleuth index files
0e52741
@iminlikewithyou
Create a permission generator
c06b76f 
Would be beneficial to have this be a Builder
@iminlikewithyou
Initialize the warn command
cc1dbba
@iminlikewithyou
Remove guild IDs to simplify permissions
0624d61
@iminlikewithyou
Create new permission getters
3e515b7
@iminlikewithyou
Outline a new permission structure
607e6be
@iminlikewithyou
Implement new permissions format
652a96b
@iminlikewithyou
Implement permissions and rate limits in existing commands
f76fe8d
@iminlikewithyou
Implement Overrides
208115b
@iminlikewithyou
Add helper functions for creating overrides
a6b6808
@iminlikewithyou
Utilize new helper functions in commands
f8dba65
@iminlikewithyou
Create helpers for interacting with the command API
2912c91
@iminlikewithyou
Use wildcard stars instead of @everyone and @ALL
9c2abcd
@iminlikewithyou
Export all types in permissions.ts
83496b7
@iminlikewithyou
Reputable is reliable
611793c
@iminlikewithyou
create new Command class
23dc1b8
@iminlikewithyou
Refactor Permissions.ts and separate RateLimits
9f1dd3d
@iminlikewithyou
complete the command code
edd78f6
@iminlikewithyou
rewrite /gamble in the class format
207bc7f
@iminlikewithyou
remove the gamba command
a690eda
@iminlikewithyou
restructure Command class
64d3c21
@iminlikewithyou
fix NormalizedRateLimit type issue
ae2d45a
@iminlikewithyou
rewrite some commands in new structure
aefec76
@iminlikewithyou
refactor and finish implementation of Permissions
ebfd558
@iminlikewithyou
add more specificity to Permissions and create conversion function
f14de59
@iminlikewithyou
refactor and fix bugs
6a64995
@iminlikewithyou
update Command to new CommandData structure
192fbc0
@iminlikewithyou
add limits to command data
c43ac51
@iminlikewithyou
update commands to match new structure (again) (again)
ede2ad5
@iminlikewithyou iminlikewithyou self-assigned this Jan 12, 2024
This was linked to issues Jan 12, 2024
Turn the bots into separate modules #82
Open
All bots' presence updates can fail without retrying #91
Open
Set up database defaults when it is empty #93
Open
command cooldowns are enforced when the command fails #125
Open
@iminlikewithyou iminlikewithyou added major A feature with substantial impact to the project and requires careful deliberation or planning. priority:1 Important. Feels bad to not address. type:testing area:modules labels Jan 17, 2024
@iminlikewithyou iminlikewithyou deleted the sleuth branch January 18, 2024 00:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@iminlikewithyou iminlikewithyou

Labels
area:modules major A feature with substantial impact to the project and requires careful deliberation or planning. priority:1 Important. Feels bad to not address. type:testing
Projects
None yet
Milestone
No milestone
1 participant
@iminlikewithyou