Skip to content

Fix client printing story context#8846

Merged
carlobeltrame merged 2 commits intoecamp:develfrom
carlobeltrame:fix-client-printing-story-context
Jan 8, 2026
Merged

Fix client printing story context#8846
carlobeltrame merged 2 commits intoecamp:develfrom
carlobeltrame:fix-client-printing-story-context

Conversation

@carlobeltrame
Copy link
Member

We named our contentType "Storycontext", but the PDF building block "Story". This led to confusion on my side in #8002.

The snapshots now correctly include the storycontext texts.

I also added a snapshot test for the safety considerations overview.

@carlobeltrame carlobeltrame requested a review from a team January 8, 2026 19:56
@manuelmeister manuelmeister added the deploy! Creates a feature branch deployment for this PR label Jan 8, 2026
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

Feature branch deployment ready!

Name Link
😎 Deployment https://pr8846.ecamp3.ch/
🔑 Login test@example.com / test
🕒 Last deployed at Thu Jan 08 2026 21:03:02 GMT+0100
🔨 Latest commit 1bacb66fae807b7a4644c5894c2af448e7d9f1e9
🔍 Latest deploy log https://github.com/ecamp/ecamp3/actions/runs/20829923324/job/59841494769
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

@carlobeltrame carlobeltrame added this pull request to the merge queue Jan 8, 2026
Merged via the queue into ecamp:devel with commit 7d6d1de Jan 8, 2026
41 checks passed
@carlobeltrame carlobeltrame deleted the fix-client-printing-story-context branch January 8, 2026 20:16
This was referenced Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy! Creates a feature branch deployment for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants