Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

docs: add notification-api-v1 spec #13

Merged
merged 10 commits into from
Feb 9, 2024
Merged

Conversation

tomwwinter
Copy link
Contributor

@tomwwinter tomwwinter commented Feb 8, 2024

Closes #9

@tomwwinter tomwwinter requested a review from sleidig February 8, 2024 16:23
Copy link
Member

@sleidig sleidig left a comment

Choose a reason for hiding this comment

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

Really nice and clear!

Just added a bit around the notification event process, please check and adapt.

docs/api-specs/reporting-api-v1.yaml Outdated Show resolved Hide resolved
docs/api-specs/reporting-api-v1.yaml Outdated Show resolved Hide resolved
docs/api-specs/reporting-api-v1.yaml Show resolved Hide resolved
docs/api-specs/reporting-api-v1.yaml Show resolved Hide resolved
docs/api-specs/reporting-api-v1.yaml Outdated Show resolved Hide resolved
docs/api-specs/reporting-api-v1.yaml Show resolved Hide resolved
@tomwwinter tomwwinter merged commit 6986bb0 into main Feb 9, 2024
1 check passed
@tomwwinter tomwwinter deleted the tw/docs/notification-api branch February 9, 2024 10:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

draft notification concept (+ OpenAPI) specs to inform clients of results ready / report results changed
2 participants