Correção do envio de variáveis pelo typeboy by matheusfterra · Pull Request #1624 · EvolutionAPI/evolution-api · GitHub
Skip to content

Correção do envio de variáveis pelo typeboy#1624

Merged
DavidsonGomes merged 2 commits intoEvolutionAPI:developfrom
matheusfterra:main
Jun 23, 2025
Merged

Correção do envio de variáveis pelo typeboy#1624
DavidsonGomes merged 2 commits intoEvolutionAPI:developfrom
matheusfterra:main

Conversation

@matheusfterra
Copy link
Copy Markdown
Contributor

@matheusfterra matheusfterra commented Jun 19, 2025

Atualmente quando no fluxo do typebot tem o envio de alguma variável, a mensagem não é enviada.

Com essa correção a mensagem é enviada normalmente.

Summary by Sourcery

Bug Fixes:

  • Bind applyFormatting to the service instance when processing Typebot messages to fix variable sending

@sourcery-ai
Copy link
Copy Markdown
Contributor

sourcery-ai Bot commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey @matheusfterra - I've reviewed your changes - here's some feedback:

  • Instead of binding this.applyFormatting on each call, consider binding it once in the constructor to avoid repeated function creation.
  • You could also define applyFormatting as an arrow method in the class so it automatically retains the correct this context without needing manual binding.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Instead of binding `this.applyFormatting` on each call, consider binding it once in the constructor to avoid repeated function creation.
- You could also define `applyFormatting` as an arrow method in the class so it automatically retains the correct `this` context without needing manual binding.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

Copy link
Copy Markdown

@JrStarkmidia JrStarkmidia left a comment

Choose a reason for hiding this comment

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

Esse correção é exatamente o que preciso para os meu bots funcionarem novamente.

@DavidsonGomes DavidsonGomes changed the base branch from main to develop June 23, 2025 18:31
@DavidsonGomes DavidsonGomes merged commit a7badb9 into EvolutionAPI:develop Jun 23, 2025
1 check passed
JrStarkmidia added a commit to JrStarkmidia/evolution-api that referenced this pull request Jun 26, 2025
…ormatting-e-proteger-entrada

Corrige erro em applyFormatting com valores não estruturados

Esta correção adiciona verificação de tipo e valor falsy antes de aplicar o processamento de texto.

Correção baseada em:
EvolutionAPI#1624
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