Skip to content

fix(sample): drop legacy Notification-T/NL/v1 extension URI - #199

Merged
zhangwenjun-huawei merged 1 commit into
project-openan:devfrom
zhangwenjun-huawei:dev
Sep 18, 2026
Merged

zhangwenjun-huawei merged 1 commit into
project-openan:devfrom
zhangwenjun-huawei:dev

Conversation

@zhangwenjun-huawei

Copy link
Copy Markdown
Contributor

Description

Replace the legacy Notification-T/NL/v1 extension URI with the canonical Notification-T/v1 across the subscribe-incident and service-recovery samples:

  • subscribe-incident: ClientSampleFlow, the server flow and the AgentCard now use Notification-T/v1 (constant renamed from NOTIFICATION_T_EXTENSION_URI_NL)
  • service-recovery: remove the legacy NL alias constant and its AgentCard extension entry; the server flow accepts and reads only Notification-T/v1

Related Issue(s)

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Other (describe below)

Checklist

  • I have read the CONTRIBUTING guide
  • All commits are signed off (git commit -s) per the DCO
  • New and existing tests pass locally
  • I have added tests that prove my fix is effective or my feature works (if applicable)
  • I have added or updated documentation as needed
  • New source files include the SPDX license header

Additional Context

Replace the legacy Notification-T/NL/v1 extension URI with the canonical Notification-T/v1 across the subscribe-incident and service-recovery samples:

- subscribe-incident: ClientSampleFlow, the server flow and the AgentCard now use Notification-T/v1 (constant renamed from NOTIFICATION_T_EXTENSION_URI_NL)
- service-recovery: remove the legacy NL alias constant and its AgentCard extension entry; the server flow accepts and reads only Notification-T/v1

Signed-off-by: zhangwenjun-huawei <zhangwenjun19@huawei.com>
@zhangwenjun-huawei
zhangwenjun-huawei requested a review from a team as a code owner September 18, 2026 13:08
@zhangwenjun-huawei
zhangwenjun-huawei merged commit 34d293b into project-openan:dev Sep 18, 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.

1 participant