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.

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

Refactor for modularity #17

Open
Glowstick0017 opened this issue Oct 2, 2023 · 6 comments
Open

Refactor for modularity #17

Glowstick0017 opened this issue Oct 2, 2023 · 6 comments
Assignees
Labels
good first issue Good for newcomers hacktoberfest-accepted help wanted Extra attention is needed

Comments

@Glowstick0017
Copy link
Owner

With new features being added and old features being modified, there needs to be a focus on modularity. Components like the context menu and commands can be refactored to have a more mainstream approach to make the addition of new objects easy for those looking to contribute, easy to read for those looking to learn, and a generally more clean up to standards environment rather than chaining if/else statements. This would be a good focus for those looking to make impactful pull requests.

@Glowstick0017 Glowstick0017 added hacktoberfest-accepted help wanted Extra attention is needed good first issue Good for newcomers labels Oct 2, 2023
@vishalpatil20
Copy link

Hello @Glowstick0017 I would like to work on this, can you explain more about the changes that you need
And assign me this issue to work on it

@Glowstick0017
Copy link
Owner Author

@vishalpatil20 for the command modularity, i was thinking making the commands either a class or objects in a json file. Each would have a name, description, expected inputs, and the function it would execute rather than the if/else chain it exists as currently.

@vlazic
Copy link
Contributor

vlazic commented Oct 4, 2023

If this task is available, I could jump in and work on it. I'm not sure if @vishalpatil20 is working on it or not, so please let me know.

@vishalpatil20
Copy link

@vlazic you can go forward with the issue

@aniledev
Copy link

aniledev commented Oct 7, 2023

I’d also like to assist with this issue if possible.

@Bilal-Shahzad
Copy link
Contributor

If you can assign this to me I would greatly appreciate it! @Glowstick0017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers hacktoberfest-accepted help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants