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

feat: remove route in router #3145

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

SaltySooda
Copy link

feature: 'remove' is complex feature . This submit only gives the basic framework, and relevant details need to be improved later, such as modifying the priority of nodes

test:already make the basic test

ps:
this is my first time to submit PR to gin,I have great passion to make contribute。please tell if I did not go the right way

@SaltySooda
Copy link
Author

#776

daheige
daheige previously approved these changes May 16, 2022
ashangpeng
ashangpeng previously approved these changes Jun 24, 2022
Copy link

@ashangpeng ashangpeng left a comment

Choose a reason for hiding this comment

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

ok

@ashangpeng
Copy link

@appleboy please review this pr. thanks .

}

}

// Routes returns a slice of registered routes, including some useful information, such as:
Copy link
Member

Choose a reason for hiding this comment

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

missing all TODO tasks?

Copy link
Author

Choose a reason for hiding this comment

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

what do you mean? do I need to create "task" for each TODO ?

@thinkerou
Copy link
Member

maybe we should separate one gin-contrib, https://github.com/gin-contrib/graceful

finish TODO
@SaltySooda SaltySooda dismissed stale reviews from ashangpeng and daheige via bb8194b July 11, 2022 08:12
@SaltySooda
Copy link
Author

@appleboy finish TODO 。

@thinkerou It is all middleware in gin-contrib. you think the "REMOVE" function should be there?

finish TODO
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.

5 participants