Skip to content
This repository has been archived by the owner on Nov 23, 2022. It is now read-only.

fix: fixed recaptcha in customer satisfaction to avoid Timeout error #426

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

giuliaghisini
Copy link
Contributor

No description provided.

@pnicolli
Copy link
Member

pnicolli commented Dec 1, 2021

A prima vista non capisco il perché nello yarn.lock ci venga fissata la versione nuova di @plone/scripts e razzle o.O
Per il resto non so dire cosa comporti la modifica perché non so cosa inserisca recaptcha in pagina 😅

@giuliaghisini
Copy link
Contributor Author

A prima vista non capisco il perché nello yarn.lock ci venga fissata la versione nuova di @plone/scripts e razzle o.O Per il resto non so dire cosa comporti la modifica perché non so cosa inserisca recaptcha in pagina 😅

per lo yarn lock non te lo so dire..
ho fatto solamente una nuova release di volto-customer-satisfaction e su design-volto-theme ho fatto
yarn add collective/volto-customer-satisfaction#v0.2.7
e ha aggiornato quelle dipendenze che dici tu.

Il motivo di questa modifica, è che abbiamo tipo 4K errori su un progetto su sentry ed è colpa di recaptcha presente nella form della customer satisfaction, che al cambio della pagina genera una nuova chiave ma non smonta completamente il componente di GoogleRecaptcha. Non smontandolo completamente rimangono dei file in pagina dipendenti dalla vecchia chiave e per questo da un errore di Timeout

@giuliaghisini
Copy link
Contributor Author

A prima vista non capisco il perché nello yarn.lock ci venga fissata la versione nuova di @plone/scripts e razzle o.O Per il resto non so dire cosa comporti la modifica perché non so cosa inserisca recaptcha in pagina 😅

Anzi, mi è venuto in mente perchè è stato aggiunto @plone/script (ma non il perchè è stato aggiunto razzle).
Il motivo è che volto-customer-satisfaction è aggiornato a Volto14 (per la parte di gestione delle traduzioni), e gli addon devono avere come dipendenza @plone/scripts.

@pnicolli
Copy link
Member

pnicolli commented Dec 1, 2021

Ah ok e quindi è aggiornato ad una versione di Volto più recente di quella che c'è su io-comune. È safe sta cosa? Se lo è, per me la PR è ok.

@giuliaghisini
Copy link
Contributor Author

Ah ok e quindi è aggiornato ad una versione di Volto più recente di quella che c'è su io-comune. È safe sta cosa? Se lo è, per me la PR è ok.

si si lo è, ma non abbiamo risolto il problema di ReCaptcha, quindi adesso non lo mergiamo

@giuliaghisini giuliaghisini marked this pull request as draft December 1, 2021 11:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants