Release r1.1 with 0.1.0-rc.1 version (Fall'25 M3) by urvika-v · Pull Request #15 · camaraproject/ApplicationEndpointRegistration

urvika-v · 2025-06-24T15:43:19Z

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 M3 release candidate v0.1.0-rc.1

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

None

Changelog input

 release-note
 - Publication of Fall'25 M3 release candidate v0.1.0-rc.1

Additional documentation

This section can be blank.

docs

jgarciatovar · 2025-06-26T07:56:10Z

Apart from that small typo in comment above, LGTM.

Kevsy

LGTM

As per API versioning guidelines "For initial public API versions 0.y.z, the MINOR number MUST be included in the URL, e.g. v0.y. "

Kevsy

LGTM (I fixed the the API version in the URL)

Kevsy · 2025-06-30T14:45:49Z

-<!-- Pre-releases of this sub project are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases -->
-<!-- The latest public release is available here: https://github.com/camaraproject/ApplicationEndpointRegistration/releases/latest -->
-<!-- For changes see [CHANGELOG.md](https://github.com/camaraproject/ApplicationEndpointRegistration/blob/main/CHANGELOG.md) -->
+Pre-releases of this sub project are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases


Suggested change

Pre-releases of this sub project are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases

Pre-releases of this API repository are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases

Commented out because there are no pre-releases

Actually should this line be used, with r1.1 there is a release in this repository.

But change it a little bit (template need to be updated):
Pre-releases of this API repository are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases

Actually should this line be used, with r1.1 there is a release in this repository

Ah yes - I forgot the clarification on the Wiki: "
A pre-release is a GitHub release of an alpha or release-candidate API version. Note: the term release is also often used here but it should be clear from the context."

Thanks, and I'll update the text to state "API repository" as suggested above.

tanjadegroot

One important change to align on standard API design practice, and a few tweeks and you will be ok to go.

tanjadegroot · 2025-07-30T17:07:14Z

@@ -2,7 +2,7 @@
 openapi: 3.0.3
 info:
  title: Application Endpoints Registration


The CAMARA (and industry) recommendation is to have the resource name always as plural.
then the POST and GET operation are on the same endpoint.

Although I agree this is a bit counter intuitive, your above approach is not standard practice for REST APIs (for developers using the API), which is to use plural for the resource name.
I recommend that you group the POST and GET operations under /application-endpoint-lists (in plural)

Co-authored-by: Tanja de Groot <87864067+tanjadegroot@users.noreply.github.com>

urvika-v · 2025-07-30T18:42:44Z

@tanjadegroot Thank you for the comments. I have done the changes, could you please review.

tanjadegroot

Thanks very much for all the work, Urvika
It looks great !
/LGTM from Release Management

Kevsy

Just a few changes (see suggestions to commit above)

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

urvika-v · 2025-07-31T09:25:45Z

Thank you @Kevsy for the comments, I have updated the Test file.

Kevsy

Thanks @urvika-v - all good from codeowner review

Kevsy · 2025-07-31T09:35:49Z

@urvika-v you can proceed with the PR merge, pre-release and wiki update: or if you let me know within the next 30 minutes of this comment I can do it :)

These are the tasks:

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)

Kevsy · 2025-07-31T09:46:28Z

@urvika-v this reviewed/approved PR was closed without merging, please proceed with the tasks from my comment above (or I can do them, just let me know)

urvika-v · 2025-07-31T09:50:31Z

@Kevsy , I guess I am not code owner. The option to merge is not available for me.

Kevsy · 2025-07-31T09:52:44Z

@Kevsy , I guess I am not code owner. The option to merge is not available for me.

No problem @urvika-v , I'll go ahead. Thanks for making all the changes 👍

urvika-v added 4 commits June 24, 2025 20:31

Update application-endpoint-registration.yaml

e5fcda0

Update CHANGELOG.md

51a1ce3

Update Application-Endpoint-Registration-API-Readiness-Checklist.md

3c71b9f

Update README.md

e198b0a

urvika-v requested review from a team, FabrizioMoggio, JoseMConde, Kevsy, crissancas, gainsley, gunjald, maheshc01 and seralogar as code owners June 24, 2025 15:43

hdamker mentioned this pull request Jun 24, 2025

ApplicationEndpointRegistration r1.1 (Fall25 M3, new API) release review camaraproject/ReleaseManagement#239

Closed

20 tasks

jgarciatovar reviewed Jun 26, 2025

View reviewed changes

Comment thread documentation/API_documentation/Application-Endpoint-Registration-API-Readiness-Checklist.md Outdated

Kevsy previously approved these changes Jun 26, 2025

View reviewed changes

Kevsy reviewed Jun 26, 2025

View reviewed changes

Comment thread code/API_definitions/application-endpoint-registration.yaml Outdated

Fixed API version in URL by adding minor version

fe54405

As per API versioning guidelines "For initial public API versions 0.y.z, the MINOR number MUST be included in the URL, e.g. v0.y. "

Kevsy dismissed their stale review via fe54405 June 26, 2025 09:39

Kevsy previously approved these changes Jun 26, 2025

View reviewed changes

maheshc01 previously approved these changes Jun 27, 2025

View reviewed changes

Update application-endpoint-registration.feature

b2652b6

urvika-v dismissed stale reviews from maheshc01 and Kevsy via b2652b6 June 30, 2025 10:40

urvika-v added 2 commits June 30, 2025 16:13

Update Application-Endpoint-Registration-API-Readiness-Checklist.md

9cea715

Update Application-Endpoint-Registration-API-Readiness-Checklist.md

c436eab

Kevsy reviewed Jun 30, 2025

View reviewed changes

Comment thread CHANGELOG.md Outdated

Kevsy reviewed Jun 30, 2025

View reviewed changes

Comment thread README.md Outdated

Kevsy reviewed Jun 30, 2025

View reviewed changes

urvika-v added 3 commits July 29, 2025 17:11

Update application-endpoint-registration.yaml

aa5d6a8

Update application-endpoint-registration.yaml

2c76b7d

Update application-endpoint-registration.yaml

9fa65b6

urvika-v requested a review from tanjadegroot July 29, 2025 20:53

tanjadegroot requested changes Jul 30, 2025

View reviewed changes

urvika-v and others added 3 commits July 30, 2025 23:56

Update README.md

af4a95b

Co-authored-by: Tanja de Groot <87864067+tanjadegroot@users.noreply.github.com>

Update README.md

632f2e0

Co-authored-by: Tanja de Groot <87864067+tanjadegroot@users.noreply.github.com>

Update application-endpoint-registration.yaml

eb56148

urvika-v requested a review from tanjadegroot July 30, 2025 18:42

tanjadegroot previously approved these changes Jul 30, 2025

View reviewed changes

Kevsy reviewed Jul 31, 2025

View reviewed changes

Comment thread code/Test_definitions/application-endpoint-registration.feature Outdated

Kevsy reviewed Jul 31, 2025

View reviewed changes

Comment thread code/Test_definitions/application-endpoint-registration.feature

Kevsy reviewed Jul 31, 2025

View reviewed changes

Comment thread code/Test_definitions/application-endpoint-registration.feature Outdated

Kevsy requested changes Jul 31, 2025

View reviewed changes

Update code/Test_definitions/application-endpoint-registration.feature

4d4bb60

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

urvika-v dismissed tanjadegroot’s stale review via 4d4bb60 July 31, 2025 09:23

Update code/Test_definitions/application-endpoint-registration.feature

440328a

Co-authored-by: Kevin Smith <Kevsy@users.noreply.github.com>

fix: added apiRoot to test inputs

0962a75

Kevsy reviewed Jul 31, 2025

View reviewed changes

Comment thread code/Test_definitions/application-endpoint-registration.feature Outdated

fix: leading whitespace

723262e

Kevsy approved these changes Jul 31, 2025

View reviewed changes

urvika-v closed this Jul 31, 2025

Kevsy reopened this Jul 31, 2025

Kevsy merged commit e1239a7 into camaraproject:main Jul 31, 2025
3 checks passed

	Pre-releases of this sub project are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases
	Pre-releases of this API repository are available in https://github.com/camaraproject/ApplicationEndpointRegistration/releases

Uh oh!

Conversation

urvika-v commented Jun 24, 2025

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for reviewers:

Changelog input

Additional documentation

Uh oh!

Uh oh!

jgarciatovar commented Jun 26, 2025

Uh oh!

Kevsy left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Kevsy left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Kevsy Jun 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Kevsy Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

hdamker Jun 30, 2025

Choose a reason for hiding this comment

Uh oh!

Kevsy Jul 1, 2025

Choose a reason for hiding this comment

Uh oh!

tanjadegroot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

tanjadegroot Jul 30, 2025

Choose a reason for hiding this comment

Uh oh!

urvika-v commented Jul 30, 2025

Uh oh!

tanjadegroot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Kevsy left a comment

Choose a reason for hiding this comment

Uh oh!

urvika-v commented Jul 31, 2025

Uh oh!

Uh oh!

Kevsy left a comment

Choose a reason for hiding this comment

Uh oh!

Kevsy commented Jul 31, 2025

Uh oh!

Kevsy commented Jul 31, 2025

Uh oh!

urvika-v commented Jul 31, 2025

Uh oh!

Kevsy commented Jul 31, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Kevsy Jun 30, 2025 •

edited

Loading

tanjadegroot left a comment •

edited

Loading