Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(entrance): error when moving an entrance to another cave #975 #979

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

vmarseguerra
Copy link
Contributor

There was a typo in the translated success message, causing an error when it was displayed

Should fix #975

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-979.westeurope.azurestaticapps.net

@urien
Copy link
Contributor

urien commented Aug 20, 2024

Salut Vincent,

Pour tester j'ai voulu raccrocher le gouffre de l'Amazonie (/6651) au réseau Felix Trombe.
Dès que je clique sur le bouton d'édition j'ai la page d'erreur

@vmarseguerra
Copy link
Contributor Author

Salut, il s'agit ici d'un cas particulier, car cette entrance n'a pas de cave associé. J'ai corrigé le problème coté front, mais il y a aussi un changement à faire du côté de l'api: GrottoCenter/grottocenter-api#1305

Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://orange-rock-0d4f87503-979.westeurope.azurestaticapps.net

Copy link
Contributor

@urien urien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cette fois ci la fenêtre pour sélectionner une cave s'ouvre bien mais j'ai une erreur a la validation
Screenshot_20240821-115605

@vmarseguerra
Copy link
Contributor Author

Oui, il y a aussi un changement du côté de l'API: GrottoCenter/grottocenter-api#1305

@urien
Copy link
Contributor

urien commented Aug 21, 2024

Je te propose de pousser directement la modification sur l'API

@BTHOM
Copy link

BTHOM commented Aug 21, 2024 via email

@urien
Copy link
Contributor

urien commented Aug 22, 2024

Salut @BTHOM, si c'est possible, il est préférable de cliquer sur le lien qui conduit a cette page et de faire directement le commentaire dans la page, plutôt que de répondre au mel que tu reçois.
Ainsi tu pourras tester sur le serveur de développement qui est indiqué et qui permet de voir le résultat du travail que Vincent a fait. Je pense que tu as fait ton test directement sur grottocenter.org où la fonctionnalité n'est pas encore mise en place, en attente de test sur le serveur de développement

Copy link
Contributor

@urien urien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cela fonctionne parfaitement. J'ai fait le changement sur 2 cavités sans problème
Merci Vincent

@vmarseguerra vmarseguerra merged commit 80fa33e into develop Aug 22, 2024
5 checks passed
@vmarseguerra vmarseguerra deleted the fix-entrance-move-message-error branch August 22, 2024 07:15
@BTHOM
Copy link

BTHOM commented Aug 22, 2024

Bonjour,
Apparemment ma réponse par mail du mercredi 21 août se retrouve (automatiquement ?) sur le log ci-dessus. Tant mieux, c'est pratique.

Pour vérifier si le bug du "Puits Férié" (fiche n° 94005) a disparu, j'ai cliqué sur le seul lien azure encore actif (les autres aboutissent à une erreur 404), à savoir :
https://orange-rock-0d4f87503-978.westeurope.azurestaticapps.net/

Hélas, même sur ce site de test et après avoir vidé le cache (F5 et Ctrl+F5), le bug persiste :
je ne peux toujours pas valider la modification du nom de l'entrée en "Férié (Puits)"

Peut être s'agit il d'un bug différent de celui concernant l'affectation d'une entrée à une nouvelle cavité/ réseau, objet initial du présent log.

A suivre, sur ce fil
BTH

@urien
Copy link
Contributor

urien commented Aug 22, 2024

Bonjour @BTHOM,
Quand tu as fais ton test Vincent avait déjà mis en ligne son travail, fermé ce ticket (il est indiqué comme Merged) et fermé le serveur de développement.
Maintenant il faut tester directement sur Grottocenter

@BTHOM
Copy link

BTHOM commented Aug 22, 2024

Bonsoir,
Je viens de tester le bug "Puits Férié" sur le site Grottocenter.
C'est toujours pareil...
Voir description du bug plus haut dans ce log ou en cliquant ci-dessous :
#979 (comment)
A+
BTH
:-)

@urien
Copy link
Contributor

urien commented Aug 22, 2024

Ok, je vais faire un autre ticket

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.

La modification des propriétés de certaines entrées génère un message d'erreur
3 participants