wire-server icon indicating copy to clipboard operation
wire-server copied to clipboard

Improve API Documentation

Open mdimjasevic opened this issue 3 years ago • 1 comments

This is a WIP PR.

Still to do:

  • [ ] It is missing some Swagger instances. GHC reports the missing ones.
  • [ ] It also lacks golden tests for the federation API types.
  • [ ] It depends on PR #1923. It has been rebased on top of develop, though.
  • [ ] Some existing tests need to be fixed to reflect the updated JSON values in the federation API.

Feel free to take over while I am away.

Checklist

  • [ ] The PR Title explains the impact of the change.
  • [ ] The PR description provides context as to why the change should occur and what the code contributes to that effect. This could also be a link to a JIRA ticket or a Github issue, if there is one.
  • [ ] If HTTP endpoint paths have been added or renamed, the endpoint / config-flag checklist (see Wire-employee only backend wiki page) has been followed.
  • [ ] If a cassandra schema migration has been added, I ran make git-add-cassandra-schema to update the cassandra schema documentation.
  • [ ] changelog.d contains the following bits of information (details):
    • [ ] A file with the changelog entry in one or more suitable sub-sections. The sub-sections are marked by directories inside changelog.d.
    • [ ] If new config options introduced: added usage description under docs/reference/config-options.md
    • [ ] If new config options introduced: recommended measures to be taken by on-premise instance operators.
    • [ ] If a cassandra schema migration is backwards incompatible (see also these docs), measures to be taken by on-premise instance operators are explained.
    • [ ] If a data migration (not schema migration) introduced: measures to be taken by on-premise instance operators.
    • [ ] If public end-points have been changed or added: does nginz need un upgrade?
    • [ ] If internal end-points have been added or changed: which services have to be deployed in a specific order?

mdimjasevic avatar Dec 06 '21 09:12 mdimjasevic

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar May 10 '23 07:05 CLAassistant