Skip to content

feat(docs): api-reference-parameter-descriptions-audit#65

Open
Caneryy wants to merge 1 commit into
wraith-protocol:developfrom
Caneryy:feat/issue-26
Open

feat(docs): api-reference-parameter-descriptions-audit#65
Caneryy wants to merge 1 commit into
wraith-protocol:developfrom
Caneryy:feat/issue-26

Conversation

@Caneryy

@Caneryy Caneryy commented Jun 28, 2026

Copy link
Copy Markdown

Closes #26

Summary

  • API reference parameter descriptions audit

Test plan

  • Relevant tests pass locally
  • Manual verification on affected UI or API paths

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@Caneryy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@truthixify

Copy link
Copy Markdown
Contributor

The api-reference parameter audit work looks substantial (213 + 374 lines across endpoints + types). Two things:

  1. Conflict on api-reference/endpoints.mdx against develop:
git fetch origin
git rebase origin/develop
# Keep develop's recent endpoint additions, then re-apply your audit descriptions
git push --force-with-lease
  1. Heads up: API reference parameter descriptions audit #26 had been assigned to @emarkees in our morning batch. @emarkees — please coordinate with @Caneryy on the issue thread if you'd already started.

@Caneryy Caneryy changed the base branch from main to develop June 28, 2026 19:06
@Caneryy

Caneryy commented Jun 28, 2026

Copy link
Copy Markdown
Author

Rebased onto develop and resolved the conflict in api-reference/endpoints.mdx.

Conflict resolution:

  • Kept develop's recent endpoint additions (typescript no-check blocks, path parameter tables)
  • Preserved audit cross-references (e.g. Chat endpoint now links to ChatResponse instead of duplicating inline types)
  • types.mdx auto-merged cleanly — Federation types from develop + parameter descriptions from this PR

Also retargeted the PR base from maindevelop to match repo convention.

Ready for re-review.

@Caneryy

Caneryy commented Jun 29, 2026

Copy link
Copy Markdown
Author

@truthixify check please

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.

API reference parameter descriptions audit

2 participants