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

feat(net/goai): enhance openapi doc with responses and examples #3859

Merged
merged 32 commits into from
Oct 21, 2024
Merged

feat(net/goai): enhance openapi doc with responses and examples #3859

merged 32 commits into from
Oct 21, 2024

Conversation

UncleChair
Copy link
Contributor

@UncleChair UncleChair commented Oct 16, 2024 •
edited
Loading

Fixes #3747

  • Added tags for response status and exampls
  • Added interface EnhancedResponse for other error response generation
  • Added coresponding test case
    • github.com/gogf/gf/v2/net/goai coverage: 80.0% of statements

VoidGun and others added 17 commits October 13, 2024 13:35
@VoidGun
Feature: add change default response status code for goai
502cb82
@VoidGun
Feature: filter field with status tag in schema
a42f483
@UncleChair
Feature: add other response structure
88056c7
@VoidGun @UncleChair
Feature: add examples for response goai doc
1c107dd
@UncleChair
Feature: add response override logic
af52524
@UncleChair
Feature; add function for examples file apply
bb56a0b
@UncleChair
Feature: add examples for responses
4071e06
@UncleChair
Test: add unit test for goai enhancement for gogf#3747
faa987f
@VoidGun @UncleChair
Pref: delete useless code
ead35af
@UncleChair
Feature: add interface for response status management
020bf12
@UncleChair
Feature; add api doc generation for enhanced response interface
84b68d6
@UncleChair
Feature: add test cases
98d0f69
@UncleChair
Pref: remove unused logic
da9066f
@UncleChair
Merge branch 'master' into goai/http_status_enhance
036c40e
@VoidGun
Refactor: add response add function
3ad0769
@VoidGun
Feature: add error check for AddStatus
62c2594
@VoidGun
Fix: unnecessary leading newline
f46a693
oldme-git
oldme-git requested changes Oct 17, 2024
View reviewed changes
net/goai/goai_example.go Outdated Show resolved Hide resolved
net/goai/goai_example.go Outdated Show resolved Hide resolved
net/goai/goai_response_ref.go Outdated Show resolved Hide resolved
@UncleChair UncleChair requested a review from oldme-git October 17, 2024 15:04
gqcn
gqcn requested changes Oct 19, 2024
View reviewed changes
net/goai/goai_response.go Show resolved Hide resolved
net/goai/goai_response_ref.go Outdated Show resolved Hide resolved
net/goai/goai_response_ref.go Outdated Show resolved Hide resolved
net/goai/goai_response.go Outdated Show resolved Hide resolved
gqcn
gqcn requested changes Oct 19, 2024
View reviewed changes
net/goai/goai_response_ref.go Outdated Show resolved Hide resolved
@gqcn gqcn merged commit 555bb3f into gogf:master Oct 21, 2024
20 checks passed
@BrewTestBot BrewTestBot mentioned this pull request Nov 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wln32 wln32 wln32 approved these changes

@oldme-git oldme-git oldme-git approved these changes

@gqcn gqcn gqcn approved these changes

@houseme houseme Awaiting requested review from houseme

@huangqian1985 huangqian1985 Awaiting requested review from huangqian1985

@hailaz hailaz Awaiting requested review from hailaz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

net/goai: I want to customize the response 500 content
5 participants
@UncleChair @gqcn @oldme-git @wln32 @VoidGun