Skip to content

Conversation

@albertoramosmonagas
Copy link
Contributor

@albertoramosmonagas albertoramosmonagas commented Aug 19, 2025

What type of PR is this?

Add one of the following kinds:

  • documentation

What this PR does / why we need it:

  • ErrorInfo schema unified with CAMARA_common.yaml and event-subscription-template.yaml → reordered required fields, clarified message, status, and code descriptions.
  • Updated mandatory description for date-time string format in template files.
  • Updates to ATP: added geohash checks (scenarios 01 & 07), included missing async operation scenario, reorganized by HTTP codes, fixed sinkCredential typo, and minor corrections.

Which issue(s) this PR fixes:

Fixes #93

Special notes for reviewers:

Nothing relevant, everything comes from the latest changes in commonalities.

Changelog input

 release-note
Alignment with Commonalities r3.3 – unified ErrorInfo schema (fields reordered, clarified descriptions for message, status, and code) and updated mandatory description for date-time format and solve fixes on ATP

Additional documentation

N/A

docs

sachinvodafone
sachinvodafone previously approved these changes Aug 19, 2025
Copy link
Collaborator

@jgarciahospital jgarciahospital left a comment

Choose a reason for hiding this comment

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

LGTM

@albertoramosmonagas
Copy link
Contributor Author

@sachinvodafone @maheshc01

I made one last change to the test file with some typos. Can you check it? With the last commit, I lost the approvals.

Thank you very much in advance

Alberto

sachinvodafone
sachinvodafone previously approved these changes Aug 21, 2025
maheshc01
maheshc01 previously approved these changes Aug 21, 2025
Copy link
Collaborator

@jgarciahospital jgarciahospital left a comment

Choose a reason for hiding this comment

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

LGTM

@jgarciahospital jgarciahospital merged commit e2a8172 into camaraproject:main Aug 29, 2025
2 checks passed
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.

Alignment with Commonalities r3.3 & ATP

4 participants