Skip to content

Comments

Fix get build status#257

Merged
Ynote merged 2 commits intoprincipalefrom
fix-get-build-status
Feb 12, 2025
Merged

Fix get build status#257
Ynote merged 2 commits intoprincipalefrom
fix-get-build-status

Conversation

@Ynote
Copy link
Contributor

@Ynote Ynote commented Feb 12, 2025

Fix #237

Cette PR répare la gestion des différents cas pour getBuildStatus qu'on n'avait pas bien compris dans #243.

De ce qu'on comprends avec #237, on part donc du principe que :

  • si on dépasse le délai de 1 minute pour checker l'existence de la page et que c'est un site GitLab Pages qui renvoie une 404, on se dit que c'est un souci de compte GitLab non-vérifié.
  • si on dépasse le délai de 1 minute pour checker l'existence de la page et que ce n'est pas une page GitLab qui renvoie une 404, on considère que le statut de la page est en erreur,
  • si on est toujours dans le délai de 1 minute, on considère qu'on est encore en train de builder le site.

@Ynote Ynote merged commit b1104ab into principale Feb 12, 2025
1 check passed
@Ynote Ynote deleted the fix-get-build-status branch February 12, 2025 17:04
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.

On a un disclaimer quand on crée un site sur GitLab (et peut-être sur GitHub aussi ?) alors qu'il est bien public

2 participants