Skip to content

Conversation

hanspagel
Copy link
Member

Problem

Currently, we don’t support themes in the FastAPI integration.

Solution

This PR adds support for themes, including a whole bunch of tests.

Fixes #4157

Checklist

I've gone through the following:

  • I've added an explanation why this change is needed.
  • I've added a changeset (pnpm changeset).
  • I've added tests for the regression or new feature.
  • I've updated the documentation.

@hanspagel hanspagel self-assigned this Jul 15, 2025
@hanspagel hanspagel requested a review from marclave as a code owner July 15, 2025 13:35
Copy link

changeset-bot bot commented Jul 15, 2025

🦋 Changeset detected

Latest commit: 250fe00

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
scalar-fastapi Patch
@scalar/galaxy Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

relativeci bot commented Jul 15, 2025

#11368 Bundle Size — 2.58MiB (0%).

250fe00(current) vs fbaaa12 main#11359(baseline)

Warning

Bundle contains 2 duplicate packages – View duplicate packages

Bundle metrics  no changes
                 Current
#11368
     Baseline
#11359
No change  Initial JS 2.58MiB 2.58MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 0% 100%
No change  Chunks 1 1
No change  Assets 1 1
No change  Modules 1738 1738
No change  Duplicate Modules 0 0
No change  Duplicate Code 0% 0%
No change  Packages 168 168
No change  Duplicate Packages 2 2
Bundle size by type  no changes
                 Current
#11368
     Baseline
#11359
No change  JS 2.58MiB 2.58MiB

Bundle analysis reportBranch hanspagel/doc-2680-cannot-parse-...Project dashboard


Generated by RelativeCIDocumentationReport issue

Copy link
Contributor

@marclave marclave merged commit 926de55 into main Jul 15, 2025
37 checks passed
@marclave marclave deleted the hanspagel/doc-2680-cannot-parse-configuration-and-theme-info-to-fastapi branch July 15, 2025 13:42
@zrquan
Copy link

zrquan commented Aug 12, 2025

@hanspagel For me, this PR didn't work right unless the color configuration in the scalar_theme variable was removed.

@zrquan zrquan mentioned this pull request Aug 12, 2025
4 tasks
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.

Cannot parse configuration and theme info to FastAPI
3 participants