Skip to content

Conversation

@PedroDiez
Copy link
Contributor

@PedroDiez PedroDiez commented Jul 8, 2025

What type of PR is this?

  • correction

What this PR does / why we need it:

This PR deals with the topic raised in Issue #496, where properties with date-time format do not include mandatory text indicated by CAMARA-API-Design-Guide ins ection 2.2, as comented within the issue.

Also deals with topic raised in Issue #498 following the next resolution approach:

shorten the mandatory text (remove recommended formats):
It must follow [RFC 3339](https://datatracker.ietf.org/doc/html/rfc3339#section-5.6) and must have time zone.

Which issue(s) this PR fixes:

Fixes #496
Fixes #498

Does this PR introduce a breaking change?

  • Yes
  • No

Special notes for reviewers:

N/A

Changelog input

 Missing Mandatory Description for date-time in CAMARA_common.yaml
 Wording alignment for date-time description in: event-subscription-template.yaml, notification-as-cloud-event.yaml, CAMARA-API-Design-Guide.md and CAMARA-API-Event-Subscription-and-Notification-Guide.md 

Additional documentation

N/A

patrice-conil
patrice-conil previously approved these changes Jul 9, 2025
Copy link
Contributor

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

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

LGTM

rartych
rartych previously approved these changes Jul 9, 2025
Copy link
Contributor

@rartych rartych left a comment

Choose a reason for hiding this comment

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

LGTM

@PedroDiez PedroDiez dismissed stale reviews from rartych and patrice-conil via c881afa July 18, 2025 12:52
patrice-conil

This comment was marked as duplicate.

Copy link
Contributor

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@rartych rartych left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

correction correction in documentation Fall25

Projects

None yet

3 participants