Release r2.2 (Fall'25 M4) #62

chinaunicomyangfan · 2025-08-29T06:48:31Z

What type of PR is this?

Add one of the following kinds:

subproject management

What this PR does / why we need it:

Publication of Fall'25 M4 public release of region-device-count v0.2.0

Updates descriptions in ErrorModel

bigludo7

Hello,
Nice job ! thanks
See my review comments on behalf of the release management team.

I need also to get a confirmation from you that API description here is up-to-date.

bigludo7 · 2025-09-04T10:11:04Z

CHANGELOG.md

+## Release Notes
+
+
+This release contains the definition and documentation of


We could add public here:
This public release contains the definition....

bigludo7 · 2025-09-04T10:24:01Z

CHANGELOG.md

+* Update pattern for x-correlator in https://github.com/camaraproject/RegionDeviceCount/pull/58
+
+ API definition **with inline documentation**:
+[[YAML]](https://github.com/camaraproject/RegionDeviceCount/blob/r2.1/code/API_definitions/region-device-count.yaml)


must be r2.2 in the url

bigludo7 · 2025-09-04T10:24:13Z

CHANGELOG.md

+
+ API definition **with inline documentation**:
+[[YAML]](https://github.com/camaraproject/RegionDeviceCount/blob/r2.1/code/API_definitions/region-device-count.yaml)
+[[View it on ReDoc]](https://redocly.github.io/redoc/?url=https://raw.githubusercontent.com/camaraproject/RegionDeviceCount/r2.1/code/API_definitions/region-device-count.yaml&nocors)


must be r2.2 in the url

CHANGELOG.md

code/API_definitions/region-device-count.yaml

code/Test_definitions/region-device-count.feature

chinaunicomyangfan · 2025-09-08T10:18:23Z

Hello, Nice job ! thanks See my review comments on behalf of the release management team.

I need also to get a confirmation from you that API description here is up-to-date.

@bigludo7 Thanks a lot for the review ,I think I have updated the file based on your comments , and I believe the API description is up-to-date

hdamker

@chinaunicomyangfan Please address #64 in a short PR before the release.

chinaunicomyangfan · 2025-09-09T02:44:42Z

@chinaunicomyangfan Please address #64 in a short PR before the release.

@hdamker Thanks for the comment,I have add the notificationsBearerAuth definition here

bigludo7 · 2025-09-09T06:59:22Z

HI @chinaunicomyangfan
Thanks for the quick update.

Few additional requests:

Missing the Mandatory template for info.description as defined here. My suggestion is to add it but also please keep what you have proposed:

# Authorization and authentication

The "Camara Security and Interoperability Profile" provides details of how an API consumer requests an access token. Please refer to Identity and Consent Management (https://github.com/camaraproject/IdentityAndConsentManagement/) for the released version of the profile.

The specific authorization flows to be used will be agreed upon during the onboarding process, happening between the API consumer and the API provider, taking into account the declared purpose for accessing the API, whilst also being subject to the prevailing legal framework dictated by local legislation.

In cases where personal data is processed by the API and users can exercise their rights through mechanisms such as opt-in and/or opt-out, the use of three-legged access tokens is mandatory. This ensures that the API remains in compliance with privacy regulations, upholding the principles of transparency and user-centric privacy-by-design.

Region Device Count API does not involve the input and output of user personal data.Therefore, the access to Region Device Count APII is defined as `Client Credentials - 2-legged`. Please refer to [Identify and Consent Management](https://github.com/camaraproject/IdentityAndConsentManagement/) for the latest detailed specification of this authentication/authorization flow.

Test file uses incorrect API name regionDeviceCount (should be region-device-count) - as defined here
region-device-count: Scope Naming - Scope name should follow pattern api-name:[resource:]action: region-device-count (minor but very easy to fix) - as defined here

Thanks again for your contribution :)

chinaunicomyangfan · 2025-09-09T10:24:32Z

@bigludo7
Thank you for your suggestion. I have fixed the first two problems, but I haven't found the last one where the scope should be modified. Can you give me some guidance.

region-device-count: Scope Naming - Scope name should follow pattern api-name:[resource:]action: region-device-count (minor but very easy to fix)

bigludo7 · 2025-09-09T13:54:53Z

@chinaunicomyangfan Sure !
This is the line 185 - very small change required.
You have - region-device-count but the guideline requires for the scope value to use pattern api-name:[resource:]action
so you should have a scope valued to - region-device-count:count

bigludo7

Approved on behalf of Release Management 👏

Next steps for the team:
• [ ] PR merged (by API repository codeowner)
• [ ] Release created within GitHub (by API repository codeowner)
• [ ] Release Tracker updated (with creation date of the release and the release tag link)

chinaunicomyangfan · 2025-09-12T02:07:05Z

@bigludo7 To align with the API description on Wiki. I have modified the API description in the Readme file in this PR. Please review again.

hdamker

Please revert all changes of github.com and githubusercontent.com domains back to the official domains.

Just seen that it was already corrected in e6c706b. Sorry for the noise.

hdamker · 2025-09-12T07:48:05Z

code/API_definitions/region-device-count.yaml

    This functionality can be used for multiple use cases, some of the possible use cases for this API are:

-    - [Emergency Rescue](https://raw.githubusercontent.com/camaraproject/RegionDeviceCount/main/documentation/API_documentation/RDC_User_Story_Emergency_Rescue.md)
+    - [Emergency Rescue](https://raw.bgithub.xyz/camaraproject/RegionDeviceCount/main/documentation/API_documentation/RDC_User_Story_Emergency_Rescue.md)


This change need still to be reverted

Thanks,done

hdamker · 2025-09-12T07:48:18Z

code/API_definitions/region-device-count.yaml

+    - [Emergency Rescue](https://raw.bgithub.xyz/camaraproject/RegionDeviceCount/main/documentation/API_documentation/RDC_User_Story_Emergency_Rescue.md)

-    - [Outdoor Live Streaming](https://raw.githubusercontent.com/camaraproject/RegionDeviceCount/main/documentation/API_documentation/RDC_User_Story_Outdoor_Live_Streaming.md)
+    - [Outdoor Live Streaming](https://raw.bgithub.xyz/camaraproject/RegionDeviceCount/main/documentation/API_documentation/RDC_User_Story_Outdoor_Live_Streaming.md)


And this one as well.

Thanks,done

bigludo7

Thanks @hdamker for the review and @chinaunicomyangfan for the correction.

We're now go to go :)

Next steps for the team:
• [ ] PR merged (by API repository codeowner)
• [ ] Release created within GitHub (by API repository codeowner)
• [ ] Release Tracker updated (with creation date of the release and the release tag link)

Thanks

ChuanyuChen

looks good to me.

chinaunicomyangfan · 2025-09-15T08:09:28Z

@bigludo7 Thanks for the review.I have create a new public release r2.2 ,and the tracker has been updated.
Please check,thanks

bigludo7 · 2025-09-15T08:15:59Z

Everything look very good :)
Thanks @chinaunicomyangfan

chinaunicomyangfan added 6 commits August 29, 2025 14:31

Update region-device-count.yaml for m4

1ee2841

Update region-device-count.feature

895b9ef

Update CHANGELOG.md for m4

1d3026d

Update README.md

015a715

Update README.md

e26bb83

Update region-device-count-API-Readiness-Checklist.md

3063954

chinaunicomyangfan requested review from ChuanyuChen and ClownsAlbert as code owners August 29, 2025 06:48

chinaunicomyangfan changed the title ~~Fall25 m4~~ Release r2.2 (Fall'25 M4) Aug 29, 2025

chinaunicomyangfan added the Fall25 label Aug 29, 2025

chinaunicomyangfan requested a review from a team August 29, 2025 06:58

chinaunicomyangfan added 2 commits August 29, 2025 18:39

Merge branch 'main' into Fall25-M4

d1c2822

Update region-device-count.feature

d9f1839

hdamker mentioned this pull request Aug 29, 2025

RegionDeviceCount r2.2 (Fall25 M4) release review camaraproject/ReleaseManagement#316

Closed

34 tasks

Update region-device-count.yaml

430302d

Updates descriptions in ErrorModel

bigludo7 reviewed Sep 4, 2025

View reviewed changes

bigludo7 requested a review from a team September 4, 2025 10:35

chinaunicomyangfan added 3 commits September 8, 2025 18:08

Update CHANGELOG.md

3f66dfd

Update region-device-count.yaml

91159b2

Update region-device-count.feature

3642880

Update region-device-count.feature

70b1370

hdamker reviewed Sep 8, 2025

View reviewed changes

Add notificationsBearerAuth definition

2cd636f

chinaunicomyangfan added 3 commits September 9, 2025 17:29

Add info.description

44cf36c

Update region-device-count.feature

be99d7f

Update region-device-count.yaml

30711ea

Update region-device-count.feature

d2f703c

Update scope of region-device-count.yaml

8f1fe68

bigludo7 requested review from a team and removed request for a team September 10, 2025 06:28

bigludo7 previously approved these changes Sep 10, 2025

View reviewed changes

Update README.md to align with API description in wiki

1f3b603

chinaunicomyangfan dismissed bigludo7’s stale review via 1f3b603 September 12, 2025 01:44

Update README.md

e6c706b

hdamker previously requested changes Sep 12, 2025

View reviewed changes

hdamker reviewed Sep 12, 2025

View reviewed changes

Update region-device-count.yaml

6a19e25

bigludo7 approved these changes Sep 12, 2025

View reviewed changes

ChuanyuChen approved these changes Sep 15, 2025

View reviewed changes

chinaunicomyangfan merged commit c855ee8 into camaraproject:main Sep 15, 2025
2 checks passed

		## Release Notes


		This release contains the definition and documentation of

Release r2.2 (Fall'25 M4) #62

Release r2.2 (Fall'25 M4) #62

Uh oh!

Conversation

chinaunicomyangfan commented Aug 29, 2025

What type of PR is this?

What this PR does / why we need it:

Uh oh!

bigludo7 left a comment

Choose a reason for hiding this comment

Uh oh!

bigludo7 Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

bigludo7 Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

bigludo7 Sep 4, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chinaunicomyangfan commented Sep 8, 2025

Uh oh!

hdamker left a comment

Choose a reason for hiding this comment

Uh oh!

chinaunicomyangfan commented Sep 9, 2025

Uh oh!

bigludo7 commented Sep 9, 2025

Uh oh!

chinaunicomyangfan commented Sep 9, 2025

Uh oh!

bigludo7 commented Sep 9, 2025

Uh oh!

bigludo7 left a comment

Choose a reason for hiding this comment

Uh oh!

chinaunicomyangfan commented Sep 12, 2025

Uh oh!

hdamker left a comment

Choose a reason for hiding this comment

Uh oh!

hdamker Sep 12, 2025

Choose a reason for hiding this comment

Uh oh!

chinaunicomyangfan Sep 12, 2025

Choose a reason for hiding this comment

Uh oh!

hdamker Sep 12, 2025

Choose a reason for hiding this comment

Uh oh!

chinaunicomyangfan Sep 12, 2025

Choose a reason for hiding this comment

Uh oh!

bigludo7 left a comment

Choose a reason for hiding this comment

Uh oh!

ChuanyuChen left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

chinaunicomyangfan commented Sep 15, 2025

Uh oh!

bigludo7 commented Sep 15, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants