Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 7, 2025

This PR removes diagnostic code added for the ntfy notification system and relocates hint texts for better UX.

Changes Made

1. Removed Diagnostic Tools

  • ✅ Deleted diagnose-ntfy.sh script
  • ✅ Removed debug logging from send_notification() function
  • ✅ Removed notification configuration logging from boot_threads()
  • ✅ Kept only error logging for troubleshooting

2. Relocated Hint Texts to Banners

The hint texts have been moved from the note section (below queue) to prominent banners above the main action buttons:

Preview Banner

  • Preview Mode: "Dies ist eine Vorschau ohne Verbindung zum FPP." now appears in a banner above the buttons
  • Show Running: "Aktuell läuft eine Show – alle Aktionen sind gesperrt." displays in a banner when applicable
  • More prominent and noticeable for users

3. Removed Redundant Note

  • ✅ Removed mark_note() call for song requests in app.py
  • ✅ The message "Wunsch wurde hinzugefügt. Position X." is already shown via toast notification
  • ✅ Eliminates duplicate messaging

Technical Details

index.html:

  • Added two new banner elements: preview-banner and show-running-banner
  • Updated renderSampleState() to use preview banner instead of note section
  • Enhanced fetchState() to show appropriate banners based on state
  • Banners are positioned above buttons for better visibility

app.py:

  • Removed debug/diagnostic logging (4 locations)
  • Removed mark_note() call for song requests (message handled by toast)

All changes maintain backward compatibility and don't affect the notification system functionality.

Original prompt

nachdem das Notification Problem mit ntfy gelöst wurde, entferne bitte wieder alle zusätzlichen diagnose peozeduren, skripte, extra Logs usw, um die App und den Code wieder kleiner zu machen.

Zweite aufgabe besteht darin, Hinweise texte, die noch unten zwischen Plalist und Kanäle sind zu entfernen oder zu verschieben.

Hier tauch z.B. noch folgender Hinweis auf:

"Wunsch 'Let i Go ( Einskönigin )' wurde hinzugefügt. Position 1."
Dies sollte hier nicht mehr stehen, sondern über die Funktion des toast-Containers abgebildet werden.

"Dies ist eine Vorschau ohne Verbindung zum FPP."
Dieser Hinweis sollte bei Bedarf im gleichen stil wie die anderen Texte in einem kastendargestellt werden, oberhalb der großen Buttons und etwas Auffäliger.

Es gint auch einen Hinweis mit der Infomration, dass Aktuell eine Show läuft und alle Funktionen deaktiviert sind.
Dieser soll bei Bedarf ebenfalls über den großen Buttons erscheinen und etwas Auffälliger.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@TimUx
Copy link
Owner

TimUx commented Dec 7, 2025

@github-copilot retry

@TimUx
Copy link
Owner

TimUx commented Dec 7, 2025

@copilot @github-copilot retry

Co-authored-by: TimUx <7571934+TimUx@users.noreply.github.com>
@TimUx
Copy link
Owner

TimUx commented Dec 7, 2025

@copilot retry

@TimUx
Copy link
Owner

TimUx commented Dec 7, 2025

@copilot retry

@TimUx
Copy link
Owner

TimUx commented Dec 7, 2025

@copilot retry

Copilot AI and others added 2 commits December 7, 2025 10:00
Co-authored-by: TimUx <7571934+TimUx@users.noreply.github.com>
Co-authored-by: TimUx <7571934+TimUx@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 7, 2025

@copilot retry

I've completed all the requested changes:

  1. ✅ Removed diagnose-ntfy.sh diagnostic script
  2. ✅ Removed debug logging from app.py (4 locations)
  3. ✅ Moved hint texts to prominent banners above buttons:
    • "Dies ist eine Vorschau ohne Verbindung zum FPP."
    • "Aktuell läuft eine Show – alle Aktionen sind gesperrt."
  4. ✅ Removed redundant "Wunsch wurde hinzugefügt" note (already shown via toast)

The preview mode banner is now prominently displayed above the action buttons as shown in the screenshot:

Preview Banner

Final commits: 1e8dddf, 3869190

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • fpp.local
    • Triggering command: /usr/bin/python3 python3 app.py (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Remove additional diagnostic procedures and warnings Remove additional diagnostic procedures and relocate hint texts Dec 7, 2025
@TimUx TimUx marked this pull request as ready for review December 7, 2025 10:40
@TimUx TimUx merged commit decc763 into main Dec 7, 2025
@TimUx TimUx deleted the copilot/remove-diagnostics-and-text-warnings branch December 8, 2025 05:48
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.

2 participants