Skip to content

fixes custom body responses having a html extension#654

Closed
Lexedia wants to merge 1 commit intoschultek:nextfrom
Lexedia:fix/correct-ext-on-build
Closed

fixes custom body responses having a html extension#654
Lexedia wants to merge 1 commit intoschultek:nextfrom
Lexedia:fix/correct-ext-on-build

Conversation

@Lexedia
Copy link
Copy Markdown

@Lexedia Lexedia commented Nov 16, 2025

Description

This is a followup on #597 that sets an approximate extension when using a custom body response.

Type of Change

  • 🛠️ Bug fix

Ready Checklist

  • I've read the Contribution Guide.
  • In case this PR changes one of the core packages, I've modified the respective CHANGELOG.md file using
    the semantic_changelog format.
  • I updated/added relevant documentation (doc comments with ///).
  • I added myself to the AUTHORS file (optional, if you want to).

@Lexedia Lexedia requested a review from schultek as a code owner November 16, 2025 22:06
@docs-page
Copy link
Copy Markdown

docs-page Bot commented Nov 16, 2025

To view this pull requests documentation preview, visit the following URL:

docs.page/schultek/jaspr~654

Documentation is deployed and generated using docs.page.

@github-actions
Copy link
Copy Markdown

Package Version Report

The following packages have been updated:
jaspr_test : 0.21.7 -> 0.22.0
jaspr : 0.21.7 -> 0.22.0
jaspr_builder : 0.21.7 -> 0.22.0
jaspr_cli : 0.21.7 -> 0.22.0
jaspr_serverpod : 0.5.8 -> 0.6.0
jaspr_router : 0.8.0 -> 0.8.1
jaspr_content : 0.4.4 -> 0.4.5
jaspr_flutter_embed : 0.4.9 -> 0.4.10
jaspr_riverpod : 0.4.1 -> 0.4.2

@schultek
Copy link
Copy Markdown
Owner

Can you elaborate why this is needed?

You can already define Routes with an extension, e.g. Route(path: 'some/file.json', ...).

@Lexedia
Copy link
Copy Markdown
Author

Lexedia commented Nov 17, 2025

Well, it was mostly for routes without an extension set, although forcing one can also work indeed.

Tbh, it was mostly my fried brain at 2 o'clock that decided to make this PR

@schultek schultek deleted the branch schultek:next April 17, 2026 14:02
@schultek schultek closed this Apr 17, 2026
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