Skip to content

fix: make public QR viewer self-contained#1586

Merged
reloxx13 merged 2 commits into
PhotoboothProject:devfrom
reloxx13:fix/public-viewer-inline-preview
Jun 22, 2026
Merged

fix: make public QR viewer self-contained#1586
reloxx13 merged 2 commits into
PhotoboothProject:devfrom
reloxx13:fix/public-viewer-inline-preview

Conversation

@reloxx13

Copy link
Copy Markdown
Collaborator

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • Documentation update
  • Bug fix
  • New feature
  • Other, please explain:

What changes did you make? (Give an overview)

This fixes the public QR viewer when the host only exposes view.php and api/download.php, but not the usual Photobooth asset paths.

  • make view.php self-contained so it no longer depends on shared CSS/JS/font assets that 404 on the public host
  • load the preview image through api/download.php?inline=1 instead of a direct data/images/... URL
  • extend api/download.php to support inline image/video responses while keeping normal download behavior unchanged
  • use the configured event text when available and fall back to the start screen title if not

Is there anything you'd like reviewers to focus on?

Please focus on whether the standalone view.php approach is acceptable for public QR endpoints where the full asset tree is not reachable.

AI used to create this Pull Request?

AI was used to implement and describe the changes. The final diff was reviewed in the local checkout and validated with:

  • ddev php -l /var/www/html/view.php
  • ddev php -l /var/www/html/api/download.php

@reloxx13 reloxx13 merged commit 2e0a84f into PhotoboothProject:dev Jun 22, 2026
7 checks passed
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.

1 participant