Skip to content
This repository was 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
giuliaghisini wants to merge 1 commit intomasterfrom
fix_customer_satisfaction_recaptcha
Draft

fix: fixed recaptcha in customer satisfaction to avoid Timeout error#426
giuliaghisini wants to merge 1 commit intomasterfrom
fix_customer_satisfaction_recaptcha

Conversation

@giuliaghisini
Copy link
Copy Markdown
Member

No description provided.

@pnicolli
Copy link
Copy Markdown
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
Copy Markdown
Member 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
Copy Markdown
Member 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
Copy Markdown
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
Copy Markdown
Member 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