Skip to content

fix: typos and parameter naming in OpenAPI spec#5338

Closed
crtahlin wants to merge 2582 commits intoethersphere:masterfrom
crtahlin:docs/openapi-fixes
Closed

fix: typos and parameter naming in OpenAPI spec#5338
crtahlin wants to merge 2582 commits intoethersphere:masterfrom
crtahlin:docs/openapi-fixes

Conversation

@crtahlin
Copy link
Copy Markdown
Contributor

Summary

  • Fix parameter name mismatch in /bytes/{reference} HEAD: address -> reference
  • Fix parameter name mismatch in /bzz/{reference} HEAD: address -> reference
  • Fix parameter name mismatch in /chunks/{address} GET: reference -> address
  • Fix typo: integerity -> integrity in /pins/check summary
  • Fix incorrect description for /pins/check ref parameter (was copy-paste of offset description)

Test plan

  • Verify OpenAPI spec validates correctly
  • Check that generated API clients use correct parameter names

istae and others added 30 commits August 16, 2024 16:33
vicerace and others added 14 commits January 8, 2026 11:26
…rsphere#5187)

Co-authored-by: Janoš Guljaš <janos@resenje.org>
Co-authored-by: Ljubisa Gacevic <ljubisa.rs@gmail.com>
Co-authored-by: Ljubiša Gačević <35105035+gacevicljubisa@users.noreply.github.com>
Co-authored-by: Ljubisa Gacevic <ljubisa.rs@gmail.com>
- Fix parameter name mismatch in /bytes/{reference} HEAD: address -> reference
- Fix parameter name mismatch in /bzz/{reference} HEAD: address -> reference
- Fix parameter name mismatch in /chunks/{address} GET: reference -> address
- Fix typo: integerity -> integrity in /pins/check summary
- Fix incorrect description for /pins/check ref parameter
@martinconic
Copy link
Copy Markdown
Contributor

Linter is failing because the commit message(Fix typos and parameter naming in OpenAPI spec]) is not following the agreed naming conventions that we are using: https://www.conventionalcommits.org/en/v1.0.0/. Please fix the message. A valid message would be: fix: typos and parameter naming in open api spec .

@crtahlin crtahlin force-pushed the docs/openapi-fixes branch from b8d2075 to d251fb8 Compare March 13, 2026 05:15
@acud
Copy link
Copy Markdown
Contributor

acud commented Mar 18, 2026

@crtahlin closing this since it appears to have many unrelated changes and the underlying spec has already changed since this PR was opened

@acud acud closed this Mar 18, 2026
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.