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.

Sign up for GitHub

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

Jump to bottom

SUMM.AI translation not working for <p>&nbsp;</p> tags #1720

Closed
timobrembeck opened this issue Sep 28, 2022 · 0 comments · Fixed by #1721
Closed

SUMM.AI translation not working for <p>&nbsp;</p> tags #1720

timobrembeck opened this issue Sep 28, 2022 · 0 comments · Fixed by #1721
Labels
🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?)
Milestone
22Q3

Comments

@timobrembeck
Copy link
Member

Describe the Bug

Our content contains a few "strange" new lines which consist of <p>-tags and one space character.
The SUMM.AI returns the error "Please check your input text. It should not be empty." for these tags.

Steps to Reproduce

  1. Create page with <p>&nbsp;</p> tag in content
  2. Try to translate into easy German
  3. See error

Expected Behavior

The elements should just stay as they are - just a non-breaking space.

Actual Behavior

The API returns an error

Additional Information

Traceback 
Sep 28 15:31:11 ERROR   integreat_cms.summ_ai_api.summ_ai_api_client - SUMM.AI translation of <HTMLSegment (text:  )> failed because of <class 'integreat_cms.summ_ai_api.summ_ai_api_client.SummAiException'>: API error: 400 - Please check your input text. It should not be empty.
@timobrembeck timobrembeck added 🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?) labels Sep 28, 2022
@timobrembeck timobrembeck added this to the 22Q3 milestone Sep 28, 2022
@timobrembeck timobrembeck mentioned this issue Sep 28, 2022
Merged
@timobrembeck timobrembeck mentioned this issue Sep 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug Something isn't working ☺️ effort: low Should be doable in <4h 🆘 prio: urgent Needs to be resolved now(?)
Projects
None yet
Milestone
22Q3
Development

Successfully merging a pull request may close this issue.

Fix translation of empty tags with SUMM.AI API for Easy German digitalfabrik/integreat-cms
1 participant
@timobrembeck