Skip to content

Conversation

zouyx
Copy link

@zouyx zouyx commented Dec 14, 2020

1.Support Spring RequestMapping
2.HandlerUtils.requestMapping,if use .path()can not get anything when use @Requestmapping
so change to value()

@geoand
Copy link
Collaborator

geoand commented Dec 14, 2020

Hi,

Thanks for the contribution.

Can you please explain the rationale behind these changes?

@zouyx
Copy link
Author

zouyx commented Dec 15, 2020

Hi,

Thanks for the contribution.

Can you please explain the rationale behind these changes?

Relate to #115 which can not run travis CI.

Just recreate it.

@geoand
Copy link
Collaborator

geoand commented Dec 15, 2020

I see thanks.

Could you please add a test as was requested in the original PR?

@zouyx
Copy link
Author

zouyx commented Dec 15, 2020

I see thanks.

Could you please add a test as was requested in the original PR?

I think will add it in this few day

@zeitlinger
Copy link

@zouyx not sure I understand this PR correctly. Is this about changing the operation name to be something more meaningful than GET?

@zouyx
Copy link
Author

zouyx commented Apr 17, 2021

@zouyx not sure I understand this PR correctly. Is this about changing the operation name to be something more meaningful than GET?

yes, you are right. because it can not get anything when use @RequestMapping

@zeitlinger
Copy link

Fixes #143

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.

3 participants