Skip to content

NLIC-1980: Group models & packages in a license bundle - #2

Open
v-rudkovskiy wants to merge 2 commits into
gh-pagesfrom
RVA/NLIC-1980/Group_models_&_packages_in_a_license_bundle
Open

NLIC-1980: Group models & packages in a license bundle#2
v-rudkovskiy wants to merge 2 commits into
gh-pagesfrom
RVA/NLIC-1980/Group_models_&_packages_in_a_license_bundle

Conversation

@v-rudkovskiy

Copy link
Copy Markdown
Member

add bundle

Copilot AI 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.

Pull request overview

This PR extends the NetLicensing v2 API specification (v2.x/netlicensing.json) by introducing a new Bundle tag, endpoints (/bundle, /bundle/{bundleNumber}, /bundle/{bundleNumber}/obtain), and a Bundle schema definition to support grouping models/packages in a license bundle.

Changes:

  • Added a new Bundle tag with external docs.
  • Added CRUD-style endpoints for Bundles plus an “obtain bundle” operation.
  • Added a Bundle definition to the Swagger/OpenAPI definitions section.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread v2.x/netlicensing.json
Comment thread v2.x/netlicensing.json
Comment thread v2.x/netlicensing.json
Comment thread v2.x/netlicensing.json
@r-brown

r-brown commented Sep 1, 2026

Copy link
Copy Markdown
Member

@v-rudkovskiy please address copilot findings

@r-brown

r-brown commented Sep 2, 2026

Copy link
Copy Markdown
Member

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, apply the suggestion exactly.

Do not make changes beyond what is described in the linked review thread.

Co-authored-by: r-brown <1361258+r-brown@users.noreply.github.com>

Copilot AI commented Sep 2, 2026

Copy link
Copy Markdown

@copilot Fix the code for all comments in this review thread.

When a review comment includes a suggested change, appl...

Addressed all findings in 486e089.

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.

4 participants