Skip to content
This repository has been archived by the owner on Oct 13, 2022. It is now read-only.

#16: add missing path parameters #18

Merged
merged 2 commits into from Feb 22, 2020

Conversation

ibuki2003
Copy link
Member

新しく追加したところに足りてなかったようです (レビュアーさーん??)

@ibuki2003 ibuki2003 added bug Something isn't working aspida Edit Api Definition labels Feb 18, 2020
@ibuki2003 ibuki2003 self-assigned this Feb 18, 2020
@ibuki2003 ibuki2003 added this to Review in progress in ProjectManager via automation Feb 18, 2020
@ibuki2003 ibuki2003 linked an issue Feb 18, 2020 that may be closed by this pull request
ProjectManager automation moved this from Review in progress to Reviewer approved Feb 19, 2020
@su8ru su8ru moved this from Reviewer approved to Review in progress in ProjectManager Feb 19, 2020
@su8ru su8ru moved this from Review in progress to Reviewer approved in ProjectManager Feb 19, 2020
@su8ru su8ru moved this from Reviewer approved to Review in progress in ProjectManager Feb 19, 2020
Copy link
Member

@Takeno-hito Takeno-hito left a comment

Choose a reason for hiding this comment

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

l471

こことかまだエラーは出るんですけど、これはまた別プルリクエストになる予定ですか

@ibuki2003
Copy link
Member Author

そういえばそうですね、
何回も使われるパラメータをそこに入れておくものなんですが、使いどころがくると存在を忘れがち

ProjectManager automation moved this from Review in progress to Reviewer approved Feb 21, 2020
@ibuki2003 ibuki2003 merged commit 099a9df into master Feb 22, 2020
ProjectManager automation moved this from Reviewer approved to Done Feb 22, 2020
@ibuki2003 ibuki2003 deleted the bugfix/#16_missing_path_parameter branch February 22, 2020 03:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
aspida Edit Api Definition bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

Swaggerでエラーが出る
3 participants