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

write URL handler function #13

Open
ethanburmane opened this issue Sep 6, 2023 · 1 comment
Open

write URL handler function #13

ethanburmane opened this issue Sep 6, 2023 · 1 comment
Assignees
Labels
high priority essential feature for the project

Comments

@ethanburmane
Copy link
Owner

No description provided.

@ethanburmane ethanburmane converted this from a draft issue Sep 6, 2023
@ethanburmane ethanburmane added the high priority essential feature for the project label Sep 6, 2023
@JNicho206
Copy link
Collaborator

My opinion on this is that having classes "GitHubHandler" and "NPMHandler" depending on the url would work well. In that case the URL handler could be an argument passed to the handler and would be acted upon inside there.

Another solution could be to have a base class "Handler" that the two above mentioned classes extend, which can implement a generic URL handler.

Whether or not we do it one of those ways doesn't really matter as much as considering what will surround the URL handler function.

@ethanburmane ethanburmane moved this from Todo to In Progress in ECE 461 Project Board Sep 7, 2023
@ethanburmane ethanburmane moved this from In Progress to Done in ECE 461 Project Board Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority essential feature for the project
Projects
Development

No branches or pull requests

3 participants