Skip to content

Add .html-postfix when building for production#103

Merged
sjockers merged 3 commits into
mainfrom
feat-102
Jul 29, 2025
Merged

Add .html-postfix when building for production#103
sjockers merged 3 commits into
mainfrom
feat-102

Conversation

@sjockers
Copy link
Copy Markdown
Contributor

Closes #102

@@ -1,4 +1,7 @@
<script lang="ts">
import { dev } from '$app/environment';
import { assets } from '$app/paths';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

see above

<script module>
import { defineMeta } from '@storybook/addon-svelte-csf';
import { within, expect } from 'storybook/test';
import { assets } from '$app/paths';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

assets ist deprecated, können wir das tauschen wir hier vorgeschlagen? https://svelte.dev/docs/kit/$app-paths#assets

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dafür müssten wir SvelteKit aktualisieren, wollte ich für diesen Change eigentlich vermeiden. Aber ich probier das mal eben.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ah ja tatsache aber ist vllt ein guter anlass

@awesomephant awesomephant self-requested a review July 29, 2025 11:54
@awesomephant
Copy link
Copy Markdown
Collaborator

lgtm smash that merge button

@sjockers sjockers merged commit 6d193e4 into main Jul 29, 2025
1 check passed
@sjockers sjockers deleted the feat-102 branch August 5, 2025 08:20
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.

ChartList: Add file extension to production links

2 participants