Skip to content

Conversation

@SjurSutterudSagen
Copy link
Contributor

This fixes a linter error with the modern linter rules, by setting the OpenAPI.base in a useEffect. Setting it directly goes against the 'rules of react', and needs to be fixed before we start using the compiler.

This fixes a linter error with the modern linter rules, by setting the
OpenAPI.base in a useEffect. Setting it directly goes against the
'rules of react', and needs to be fixed before we start using the
compiler.
@github-actions
Copy link

Branch Deployment to Cloudflare Pages

🚛 Preview URL: https://9ba4eab8.pxweb2.pages.dev
🚧 Branch preview URL: https://fix-linter-error-for-setting.pxweb2.pages.dev

@sonarqubecloud
Copy link

Copy link
Contributor

@MikaelNordberg MikaelNordberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@MikaelNordberg MikaelNordberg merged commit 1eda9bd into main Nov 19, 2025
12 checks passed
@MikaelNordberg MikaelNordberg deleted the fix/linter-error-for-setting-base-api-url-directly branch November 19, 2025 09:46
MikaelNordberg added a commit that referenced this pull request Nov 19, 2025
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