Skip to content

Align ClickToDial with Commonalities r4.4 - #122

Merged
YadingFang merged 1 commit into
mainfrom
codex/commonalities-r4.4-api-alignment
Sep 14, 2026
Merged

YadingFang merged 1 commit into
mainfrom
codex/commonalities-r4.4-api-alignment

Conversation

@YadingFang

@YadingFang YadingFang commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

correction

What this PR does / why we need it:

Aligns the ClickToDial API definition and createCall tests with the Commonalities r4.4 / 0.9.0 baseline after the release-plan update in #119 and the automated common-file sync in #120.

The changes:

  • replace the mandatory additional-error-responses block with the canonical r4.4 text
  • add x-correlator to the callback request and successful callback response
  • reuse the Commonalities HTTPS-only Sink schema and remove the duplicate local Sink definition
  • replace deprecated local Generic error responses with the r4.4 reusable responses
  • retain the ClickToDial-specific 422 response and add EVENT_NOTIFICATIONS_NOT_SUPPORTED for implicit notifications
  • document that no call resource is created when notification delivery is requested but unsupported
  • add rainy-day coverage for EVENT_NOTIFICATIONS_NOT_SUPPORTED
  • update the invalid sink scenario to verify the HTTPS-only constraint
  • update x-camara-commonalities from 0.8.0 to 0.9.0

Which issue(s) this PR fixes:

N/A

@camara-validation

Copy link
Copy Markdown

CAMARA Validation — PASS

0 errors, 0 warnings, 0 hints | Profile: standard

View full results

@YadingFang
YadingFang marked this pull request as ready for review September 14, 2026 02:31

@Wojiaozhenghao Wojiaozhenghao left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ok

@YadingFang
YadingFang merged commit 5bf44bf into main Sep 14, 2026
3 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.

2 participants