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

feat(*): Revamp Handler #34

Closed
wants to merge 139 commits into from
Closed

feat(*): Revamp Handler #34

wants to merge 139 commits into from

Conversation

EvolutionX-10
Copy link
Member

No description provided.

jacoobes added 30 commits March 8, 2022 11:43
@jacoobes
feat(sern.ts) attempting to remove the bloated class in favor of smaller
cfb7df8 
functions, install rxjs
@jacoobes
refactor(handler) change init param to Wrapper, add eventConfig type
84fc853
@jacoobes
feat(sern.ts) Add basic event manager
0dd95be
@jacoobes
docs(wrapper.ts) Add events property to JSDoc
9ce1314
@jacoobes
feat(handler) new ready event handling
08f5937 
Also added new enum errors,
removed useless promise fn sign,
and more
@jacoobes
feat(events) Moving default events to its own folder for organization
df626e0
@jacoobes
feat(handler) begin message event and wrapper.prefix =
10aee77 
wrapper.defaultPrefix
@jacoobes
feat(handler) Filters only executable messages now
6d2af47
@jacoobes
refactor(sern.ts) Slowly change handler class to init function
cdd8fce
@jacoobes
build(logger.ts) Remove day.js dependency in favor of native
0513391
@jacoobes
feat(module.ts) Removing and minimizing module. Will allow simplication
30b7f59 
of revamping
@jacoobes
feat(command.ts) Add command class
f1ffff4
@jacoobes
feat(handler) finish loading command functionality, adjust properties
ecf7ecc 
per cmd type
@jacoobes
feat(command.ts) throw error on alias present on slash command
840619d
@jacoobes
feat(handler) create command-options interface, remove module
5b968ce
@jacoobes
refactor(command.ts) properties no longer look as ugly
586d386
@jacoobes
feat : refactor command objects. Deciding not to use a class
5a1870b
@jacoobes
feat (handler) moving to dev build; new module types
3178f18
@jacoobes
feat(commands) create module handler to deal with incoming data
4102c71
@jacoobes
feat : update module loader for extensibility
5b53d42
@jacoobes
feat (handler) more progress on message handler
2ea3558
@jacoobes
feat : finish basic message event handler
1ad429f
@jacoobes
feat(handler) added module type checking and started interactions han…
99aae7d 
…dler
@jacoobes
feat : remove parse option, allowing user to custom parse arguments
05fadf0
@jacoobes
feat(types) added util type Override
5baff09
@jacoobes
feat(handler) add basic interaction handling and add error for not
d34c188 
detecting modules
@jacoobes
feat(interactionCreate.ts) finish basic interaction handling for chat…
a227f1a 
… input commands
@jacoobes
feat(handler) improving context api
44e6e58
@jacoobes
refactor : modify module typings to override
2191fda
@jacoobes
refactor : exports now simplified, removed default arg parsers as parse
c002c27 
method removed
@EvolutionX-10 EvolutionX-10 marked this pull request as ready for review May 20, 2022 04:32
@EvolutionX-10 EvolutionX-10 changed the title RevampHandler feat(*RevampHandler May 20, 2022
@EvolutionX-10 EvolutionX-10 changed the title feat(*RevampHandler feat(*): Revamp Handler May 20, 2022
@Murtatrxx
Copy link
Member

Murtatrxx commented May 20, 2022 •
edited
Loading

Keep in mind, conflicts should resolved after revamp-handler is successfully merged with revamp-docs (#47) and #47 must merged before this.

jacoobes added 17 commits May 21, 2022 23:54
@jacoobes
feat: add externallyUsed.ts and support BothCommands again
fc81bfc
@jacoobes
feat: remove comments about prev commit
a220949
@jacoobes
style: change prettier print width and reformat
2675607
@jacoobes
feat: adding modal and autocomplete support
77856ce
@jacoobes
feat: Finishing up autocomplete, need to test
d50b801
@jacoobes
feat: Making name required in auto cmp interactions
ac8a2f4
@jacoobes
feat: move name and description out of OptionsData[]
93942bd
@jacoobes
feat: finishing optionData for autocomplete changes, adding class for…
b08eebf 
… builder
@jacoobes
feat: finishing autocomplete!!
d63423c
@jacoobes
refactor: Simplifying logic of message filter
9eef9dd
@jacoobes
feat: Adding docs to some data structures, moving to default from exp…
0ae541d 
…ort files
@jacoobes
refactor: cleaning up code, removing unuseds
e69dae9
@jacoobes
feat: adding refactoring for repetitive event plugin processing
475b073
@jacoobes
feat: adding better typings, refactoring
99e2a99
@jacoobes
refactor: remove unnecessary nullish short circuit
ae59aca
@jacoobes
refactor: remove unnecessary nullish short circuit
cfd68b4
@jacoobes
feat: add docs/ to npmignore
f90342d
@EvolutionX-10
Copy link
Member Author

@jacoobes Closing the PR due to the discussion and solution mentioned in Discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Murtatrxx Murtatrxx Murtatrxx requested changes

@Allyedge Allyedge Allyedge approved these changes

@jacoobes jacoobes Awaiting requested review from jacoobes jacoobes is a code owner

@trueharuu trueharuu Awaiting requested review from trueharuu

Assignees

@jacoobes jacoobes

@Murtatrxx Murtatrxx

@EvolutionX-10 EvolutionX-10

Labels
meta: Enhancement New feature or request
Projects
None yet
Milestone
v1.0.0
Development

Successfully merging this pull request may close these issues.
5 participants
@EvolutionX-10 @Murtatrxx @Allyedge @trueharuu @jacoobes