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.

Sign up for GitHub

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

Jump to bottom

希望hertz增加多一点的shortcut API，保持API易用性 #946

Closed
xj524598 opened this issue Sep 16, 2023 · 4 comments
Closed

希望hertz增加多一点的shortcut API，保持API易用性 #946

xj524598 opened this issue Sep 16, 2023 · 4 comments
Assignees
@chaoranz758
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@xj524598
Copy link

举例：RequestHeader 只有Delbytes这个API，而ResponseHeader却有Del这个API。
Del的API显然更适合大部分的业务场景，此外两个Header的API不一致，也会让用户迷惑
@github-actions github-actions bot added the invalid issue invalid issue (not related to Hertz or described in document or not enough information provided) label Sep 16, 2023
@li-jin-gou li-jin-gou added enhancement New feature or request good first issue Good for newcomers and removed invalid issue invalid issue (not related to Hertz or described in document or not enough information provided) labels Sep 16, 2023
@welkeyever
Copy link
Member

welkeyever commented Sep 18, 2023 •
edited
Loading

  • 给RequestHeader补充一个.Del API（参考ResponseHeader）

@chaoranz758
Copy link
Member

please asign to me

@welkeyever
Copy link
Member

please asign to me

@chaoranz758 It's yours👏

@chaoranz758 chaoranz758 mentioned this issue Sep 18, 2023
Merged
3 tasks
@welkeyever
Copy link
Member

@xj524598 Related PR has been merged.
Let me know if there are any new. Otherwise, I will go ahead and close this pull request for now since there don't seem to be any other available tasks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chaoranz758 chaoranz758

Labels
enhancement New feature or request good first issue Good for newcomers
Milestone
No milestone
Development

No branches or pull requests
4 participants
@welkeyever @xj524598 @li-jin-gou @chaoranz758