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

🐛 fix IndyAttrValue bad reference in OpenAPI spec #3090

Merged

Conversation

ff137
Copy link
Contributor

@ff137 ff137 commented Jul 8, 2024

Resolves #3086

See comment for detail: #3086 (comment)


You'll see in the commit logs I first regenerated the OpenAPI spec, to capture all the latest changes since 0.12.1.
Then I upgraded the openapi gen tools to latest versions, which just improves the default model names in a couple cases:

  • IndyProofProofProofsProof_non_revoc_proof -> IndyNonRevocProof
  • IndyPrimaryProof_eq_proof -> IndyEQProof

No other changes from upgrading the openapi generators to latest versions.

Then, I removed the description metadata from the IndyAttrValue field in DictWithIndyAttrValueSchema, and regenerated the spec. That solves the bad reference.

ff137 added 4 commits July 8, 2024 15:38
Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: ff137 <ff137@proton.me>
Signed-off-by: ff137 <ff137@proton.me>
…openapi specs

Signed-off-by: ff137 <ff137@proton.me>
Copy link

sonarcloud bot commented Jul 8, 2024

@dbluhm dbluhm merged commit be7b870 into openwallet-foundation:main Jul 8, 2024
8 checks passed
@ff137 ff137 deleted the fix/IndyAttrValue-bad-reference branch July 8, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OpenAPI: Bad reference
2 participants