Skip to content

feat: Add package info to some responses#707

Open
ffflorian wants to merge 8 commits into
mainfrom
feat/packageinfo
Open

feat: Add package info to some responses#707
ffflorian wants to merge 8 commits into
mainfrom
feat/packageinfo

Conversation

@ffflorian
Copy link
Copy Markdown
Owner

@ffflorian ffflorian commented Sep 4, 2021

No description provided.

@ffflorian ffflorian force-pushed the main branch 2 times, most recently from 229e109 to 7297a70 Compare March 15, 2022 20:22
@ffflorian
Copy link
Copy Markdown
Owner Author

@copilot resolve conflicts

Co-authored-by: ffflorian <5497598+ffflorian@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

@copilot resolve conflicts

Resolved in 8148458. I merged latest main, resolved the packagesRoute modify/delete conflict by keeping the new controller-based structure, and carried over the package-info response behavior into src/controllers/packages.controller.ts.

@ffflorian
Copy link
Copy Markdown
Owner Author

@copilot update swagger

Copilot AI and others added 2 commits May 13, 2026 10:28
Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

@copilot update swagger

Done in 7e3ad00 (plus 25e57c5). Swagger now documents packageInfo on package-route 404 JSON responses, updates error examples to use message, and clarifies when packageInfo is returned.

@ffflorian
Copy link
Copy Markdown
Owner Author

@copilot merge from main and include the new swagger generation method (no static file anymore)

# Conflicts:
#	swagger.json

Co-authored-by: ffflorian <5497598+ffflorian@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

@copilot merge from main and include the new swagger generation method (no static file anymore)

Done in 6fe05c2. I merged latest main, removed the static swagger.json, and switched fully to the generated Swagger method from src/swagger.ts/decorators. Also kept packageInfo documented for the package 404 response in the generated schema. Screenshot: N/A (no UI change; backend/OpenAPI generation only).

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