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

refactor: helper method for register swagger api #12352

Closed
wants to merge 12 commits into from
Closed

refactor: helper method for register swagger api #12352

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
f5db11e
added re-usable helper method for RegisterSwaggerAPI
deepto98 Jun 24, 2022
b42e3e9
Merge remote-tracking branch 'origin/main' into deepto98/helper-metho…
deepto98 Jun 24, 2022
d037036
moved RegisterSwaggerAPI from baseapp to server, added changelog
deepto98 Jul 3, 2022
409157b
removed unused import from spi/server.go, changed swagger file name t…
deepto98 Jul 4, 2022
0b1d51c
fixed comment
deepto98 Jul 4, 2022
84a1dc9
fixed changelog message
deepto98 Jul 10, 2022
0e15678
fixed conflicts
deepto98 Jul 10, 2022
03375ec
fix
deepto98 Jul 10, 2022
2e11b03
Update server/swagger.go
deepto98 Jul 10, 2022
6ee25b6
Update CHANGELOG.md
julienrbrt Jul 13, 2022
896dfa2
Update simapp/app.go
deepto98 Jul 18, 2022
229a17a
Merge branch 'main' into deepto98/helper-method-for-RegisterSwaggerAPI
julienrbrt Jul 21, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
moved RegisterSwaggerAPI from baseapp to server, added changelog
  • Loading branch information
@deepto98
deepto98 committed Jul 3, 2022
commit d03703698050d2185fdb4439701ac7e7ca65b800
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
# Changelog

## [Unreleased]

* (server) [\#12352](https://github.com/cosmos/cosmos-sdk/pull/12352) Move the `RegisterSwaggerAPI` logic into a separate helper function in the server package
amaury1093 marked this conversation as resolved.
Show resolved Hide resolved
### Features

* (cli) [#12028](https://github.com/cosmos/cosmos-sdk/pull/12028) Add the `tendermint key-migrate` to perform Tendermint v0.35 DB key migration.
Expand Down
2 changes: 1 addition & 1 deletion baseapp/register_swagger_api.go → server/register_swagger_api.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
package baseapp
package server

import (
"net/http"
Expand Down
3 changes: 2 additions & 1 deletion simapp/app.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import (
"cosmossdk.io/core/appconfig"

"github.com/cosmos/cosmos-sdk/depinject"
deepto98 marked this conversation as resolved.
Show resolved Hide resolved
"github.com/cosmos/cosmos-sdk/server"
Copy link
Member

Choose a reason for hiding this comment

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

Can you group the imports :)


"github.com/cosmos/cosmos-sdk/baseapp"
"github.com/cosmos/cosmos-sdk/codec"
Expand Down Expand Up @@ -408,7 +409,7 @@ func (app *SimApp) RegisterAPIRoutes(apiSvr *api.Server, apiConfig config.APICon
app.App.RegisterAPIRoutes(apiSvr, apiConfig)

// register swagger API (using common function from baseapp package) from root so that other applications can override easily
baseapp.RegisterSwaggerAPI(apiSvr.ClientCtx, apiSvr.Router, apiConfig.Swagger)
server.RegisterSwaggerAPI(apiSvr.ClientCtx, apiSvr.Router, apiConfig.Swagger)
}

// GetMaccPerms returns a copy of the module account permissions
Expand Down