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: remove unused endpoints #566

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Conversation

murilx
Copy link
Contributor

@murilx murilx commented Nov 21, 2024

  • Remove /tree/tests and /tree/<commit_hash>/tests that were no longer being used, including the views for these endpoints in the backend and the calling of these endpoints in the frontend.

Closes #556

Copy link
Collaborator

@WilsonNet WilsonNet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job

Copy link
Collaborator

@MarceloRobert MarceloRobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can also delete the frontend route for /tree/<treeId>/test since you're removing it from the backend

- Remove `/tree/tests` and `/tree/<commit_hash>/tests` that were no
longer being used, including the views for these endpoints in the
backend and the calling of these endpoints in the frontend, including
the route for it.

Closes #556
@murilx murilx force-pushed the refactor/remove-unused-endpoints branch from 69f0134 to ff9c2c5 Compare November 22, 2024 12:46
Copy link
Collaborator

@MarceloRobert MarceloRobert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@murilx murilx merged commit 4cf9376 into main Nov 22, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove unused endpoints
3 participants