Skip to content

fix: resolve extended text box save issue when question title has a c…#2

Open
liviacarolgouvea wants to merge 1 commit intolepidus:mainfrom
liviacarolgouvea:fix/custom-question-textarea-comma
Open

fix: resolve extended text box save issue when question title has a c…#2
liviacarolgouvea wants to merge 1 commit intolepidus:mainfrom
liviacarolgouvea:fix/custom-question-textarea-comma

Conversation

@liviacarolgouvea
Copy link
Copy Markdown

The form's fieldName generation was based on the custom question's title.
If the title contained a comma, it would generate an invalid HTML ID/CSS selector that caused the TinyMCE initialization to silently fail in the frontend, preventing the value from being saved.

Updated fieldName to use a static and safe prefix ('customQuestion-') alongside the custom question ID.

@diegoabadan
Copy link
Copy Markdown
Member

Obrigado, @liviacarolgouvea !

Como essa mudança quebra os testes cypress atuais, vamos precisar mudar também os testes. E há a questão de compatibilidade das questões salvas em instalações correntes, com os nomes da forma atual. Em breve esperamos lançar a versão nova com as correções.

@thiagolepidus
Copy link
Copy Markdown
Member

Olá @liviacarolgouvea,

Lançamos uma nova versão do plugin a correção do nome das perguntas no formulário: v1.1.0.1.

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