Skip to content

fix(docs): Incorrectly mark Path Parameters as Query Parameters#41

Merged
sid-bruno merged 3 commits into
opencollection-dev:mainfrom
sachin-bruno:sachin-bruno/fix-incorrect-path-paramters
Jun 12, 2026
Merged

fix(docs): Incorrectly mark Path Parameters as Query Parameters#41
sid-bruno merged 3 commits into
opencollection-dev:mainfrom
sachin-bruno:sachin-bruno/fix-incorrect-path-paramters

Conversation

@sachin-bruno

@sachin-bruno sachin-bruno commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Problem
The interactive HTML documentation Bruno generates for a collection renders requests with Path parameters incorrectly in two related ways:

Path parameters appear under a "Query Parameters" heading — there is no Path Parameters section.

The displayed URL keeps its raw :placeholder segments, and the page's Try runner sends those segments verbatim. The server therefore receives GET /users/:username instead of GET /users/Billy, and echoes the literal placeholder back in the response ("Hello :username!").

Bruno itself handles the same request correctly inside the app (response: {"message": "Hello Billy!"}); only the generated documentation is wrong. Anyone consuming the docs is misled into thinking the API takes a query parameter named username instead of a path segment.

https://usebruno.atlassian.net/browse/BRU-3172

@sachin-bruno sachin-bruno force-pushed the sachin-bruno/fix-incorrect-path-paramters branch from 6e15ad2 to 23dfe1f Compare June 11, 2026 06:50

@sundram-bruno sundram-bruno 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.

LGTM

@sid-bruno sid-bruno changed the title fix(size/L): Collection Docs: Incorrectly mark Path Parameters as Query Parameters fix(docs): Incorrectly mark Path Parameters as Query Parameters Jun 12, 2026
@sid-bruno sid-bruno merged commit 3fc6f13 into opencollection-dev:main Jun 12, 2026
1 check 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.

3 participants