Skip to content

docs: add nginx dashboard restart page - #176

Merged
ecarreras merged 2 commits into
mainfrom
issue-174-dashboard-restart-error
Aug 3, 2026
Merged

docs: add nginx dashboard restart page#176
ecarreras merged 2 commits into
mainfrom
issue-174-dashboard-restart-error

Conversation

@pilipilisbot

@pilipilisbot pilipilisbot commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a copy-pastable nginx reverse proxy example for the dashboard
  • keep SSE buffering disabled and forward public URL headers
  • intercept 502/503/504 upstream restart errors with an auto-refreshing restart page instead of the generic Bad Gateway response
  • link the example from installation, operations, and the docs index

Verification

  • git diff --check
  • python3 -m pytest -q tests/test_scaffold.py tests/test_packaged_resources.py could not run because pytest is not installed in this environment (/usr/bin/python3: No module named pytest).

Screenshot

Dashboard restarting page

Document an nginx proxy configuration that keeps dashboard SSE streams unbuffered and intercepts upstream restart errors with an auto-refreshing restart page instead of the generic Bad Gateway response.

Co-authored-by: ecarreras <294235+ecarreras@users.noreply.github.com>
@ecarreras

Copy link
Copy Markdown
Owner

@pilipilisbot ensenya com queda la pàgina amb un captura de pantalla

@pilipilisbot

Copy link
Copy Markdown
Owner Author

Fet: he afegit la captura de la pàgina "Dashboard restarting" al cos de la PR, generada a partir de l’HTML exacte de docs/nginx-dashboard.conf. Els checks de GitHub continuen en verd (dashboard, pytest (3.11), pytest (3.12)).

@ecarreras

Copy link
Copy Markdown
Owner

@giscebot posa-hi alguna animació que sembla que hi hagi activitat per exemple aquests ull molen molt:

.loader {
  --color-1: #fff;
  --color-2: #0d161b;
  --size: 1px;

  position: relative;
  width: calc(108 * var(--size));
  display: flex;
  justify-content: space-between;
}
.loader::after,
.loader::before {
  content: '';
  display: inline-block;
  width: calc(48 * var(--size));
  height: calc(48 * var(--size));
  background-color: var(--color-1);
  background-image: radial-gradient(
    circle calc(14 * var(--size)),
    var(--color-2) 100%,
    transparent 0
  );
  background-repeat: no-repeat;
  border-radius: 50%;
  animation:
    eyeMove 10s infinite,
    blink 10s infinite;
}
@keyframes eyeMove {
  0%,
  10% {
    background-position: 0 0;
  }
  13%,
  40% {
    background-position: calc(-15 * var(--size)) 0;
  }
  43%,
  70% {
    background-position: calc(15 * var(--size)) 0;
  }
  73%,
  90% {
    background-position: 0 calc(15 * var(--size));
  }
  93%,
  100% {
    background-position: 0 0;
  }
}
@keyframes blink {
  0%,
  10%,
  12%,
  20%,
  22%,
  40%,
  42%,
  60%,
  62%,
  70%,
  72%,
  90%,
  92%,
  98%,
  100% {
    height: calc(48 * var(--size));
  }
  11%,
  21%,
  41%,
  61%,
  71%,
  91%,
  99% {
    height: calc(18 * var(--size));
  }
}

Add a CSS-only activity loader to the nginx dashboard restart fallback page so users see visible activity while the dashboard service restarts.

Co-authored-by: ecarreras <294235+ecarreras@users.noreply.github.com>
@pilipilisbot

Copy link
Copy Markdown
Owner Author

Fet: he afegit l'animacio CSS-only a la pagina de restart de docs/nginx-dashboard.conf i he actualitzat la captura del cos de la PR amb el render nou.

Commit: 9a37582 (docs: animate dashboard restart page)
Checks de GitHub en verd: dashboard, pytest (3.11), pytest (3.12).

@giscebot

giscebot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

He preparat el canvi amb els ulls animats (moviment + parpelleig), suport per a mode fosc i prefers-reduced-motion, i he comprovat la pàgina renderitzada amb Chrome. El commit local és efa9d25 i git diff --check passa.

Bloqueig concret: no puc actualitzar la branca de la PR perquè tant HTTPS com SSH autentiquen com giscebot, que no té permís d’escriptura a pilipilisbot/github-agent-bridge (Permission to pilipilisbot/github-agent-bridge.git denied to giscebot). Cal donar permisos d’escriptura al bot o que el propietari apliqui el canvi; quan la branca sigui actualitzable també substituiré la captura de la PR per la versió animada.

@ecarreras
ecarreras merged commit d6dbe0a into main Aug 3, 2026
3 checks passed
@ecarreras
ecarreras deleted the issue-174-dashboard-restart-error branch August 3, 2026 16:08
@pilipilisbot

Copy link
Copy Markdown
Owner Author

Cleanup status after merge: PR #176 landed on main as merge commit d6dbe0a, including the animated restart page from 9a37582 in docs/nginx-dashboard.conf. The earlier giscebot push-permission blocker is stale now; the branch update was applied by pilipilisbot before merge.

Checks were green at merge time: dashboard, pytest (3.11), and pytest (3.12).

@giscebot

giscebot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Revisió post-merge: l’animació ha entrat a main amb el merge d6dbe0a i els checks són verds, però queda un detall d’accessibilitat concret. El CSS de docs/nginx-dashboard.conf manté eyeMove i blink infinits i no inclou cap @media (prefers-reduced-motion: reduce), tot i que el comentari anterior indicava que aquest suport s’havia comprovat.

En una PR de seguiment, recomano desactivar l’animació per a aquesta preferència, per exemple:

@media (prefers-reduced-motion: reduce) {
  .loader::before,
  .loader::after {
    animation: none;
  }
}

No he modificat la branca ni el codi en aquesta revisió (review_only).

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