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: add integrated swagger for finschia #223

Merged
merged 5 commits into from
Jun 23, 2023

Conversation

zemyblue
Copy link
Member

@zemyblue zemyblue commented Jun 22, 2023

Description

closes: #214

Add integrated swagger host.
The query or service proto of finschia-sdk and wasmd and ibc-go are integrated.

And I add a check ci in order to konw if the protobuf of module is changed. (check-generated.yml)

This swagger can be updated by make proto-swagger-gen.

Motivation and context

The swagger interface of current finschia node display only fisnchia-sdk APIs, so we cannot use wasmd's api and ibc-go's api. It is very uncomfortable.

How has this been tested?

  1. Set swagger enable in app.toml.
  2. Execute fnsad
  3. Connect domain::1317/swagger/#/

Screenshots (if appropriate):

스크린샷 2023-06-22 오후 9 26 26
스크린샷 2023-06-22 오후 9 27 51

Checklist:

  • I followed the contributing guidelines and code of conduct.
  • I have added a relevant changelog to CHANGELOG.md
  • I have added tests to cover my changes.
  • I have updated the documentation accordingly.

Signed-off-by: zemyblue <zemyblue@gmail.com>
@zemyblue zemyblue added the enhancement New feature or request label Jun 22, 2023
@zemyblue zemyblue self-assigned this Jun 22, 2023
Signed-off-by: zemyblue <zemyblue@gmail.com>
@codecov
Copy link

codecov bot commented Jun 22, 2023

Codecov Report

Merging #223 (89351b6) into main (21dd2bd) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #223   +/-   ##
=======================================
  Coverage   43.97%   43.97%           
=======================================
  Files           8        8           
  Lines        1087     1087           
=======================================
  Hits          478      478           
  Misses        591      591           
  Partials       18       18           
Impacted Files Coverage Δ
app/app.go 84.06% <ø> (ø)

@zemyblue zemyblue marked this pull request as draft June 22, 2023 03:44
Signed-off-by: zemyblue <zemyblue@gmail.com>
Signed-off-by: zemyblue <zemyblue@gmail.com>
Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit b98ef07008b1a1e93882447e67286a0a2ae6339f)
@zemyblue zemyblue added this to the v2.0.0 milestone Jun 22, 2023
@zemyblue zemyblue marked this pull request as ready for review June 22, 2023 11:50
Copy link
Member

@tkxkd0159 tkxkd0159 left a comment

Choose a reason for hiding this comment

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

LGTM

scripts/generate-docs.sh Show resolved Hide resolved
Makefile Show resolved Hide resolved
@zemyblue zemyblue requested a review from loin3 June 23, 2023 04:37
@zemyblue zemyblue merged commit d649d83 into Finschia:main Jun 23, 2023
@zemyblue zemyblue deleted the feat/swagger branch June 23, 2023 05:29
zemyblue added a commit to zemyblue/finschia that referenced this pull request Jun 23, 2023
* feat: add integrated swagger for finschia.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: add a ci to check generated code.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: add a ci to check generated code up-to-date

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: ci error

Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit b98ef07008b1a1e93882447e67286a0a2ae6339f)

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit d649d83)
zemyblue added a commit that referenced this pull request Jun 26, 2023
…res (#221, #223) (#225)

* feat: add integrated swagger for finschia (#223)

* feat: add integrated swagger for finschia.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: add a ci to check generated code.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: add a ci to check generated code up-to-date

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: ci error

Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit b98ef07008b1a1e93882447e67286a0a2ae6339f)

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit d649d83)

* feat: compile static binary as release assets and docker image (#221)

* build: modify docker image base and binary using static library.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: change the description in readme

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: localnet-docker-build unittest

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: change the github action ci for pushing Docker images

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: change release build for using static binary assets.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: update changelog

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: parse tag in github action of docker.yml

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: change docker build and push trigger

Signed-off-by: zemyblue <zemyblue@gmail.com>

* ci: remove condition if `pull_request` or not.

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: typo

Signed-off-by: zemyblue <zemyblue@gmail.com>

* chore: remove debian11 and nonroot base docker image

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
(cherry picked from commit 225b581)

* fix: build option error

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: build option error in docker

Signed-off-by: zemyblue <zemyblue@gmail.com>

* fix: build option error in docker

Signed-off-by: zemyblue <zemyblue@gmail.com>

---------

Signed-off-by: zemyblue <zemyblue@gmail.com>
@zemyblue zemyblue mentioned this pull request Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Need to provide integrated swagger
4 participants