Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

fix(backend): fix post language detection #6720

Merged
merged 2 commits into from
Sep 12, 2023
Merged

fix(backend): fix post language detection #6720

merged 2 commits into from
Sep 12, 2023

Conversation

Tirokk
Copy link
Member

@Tirokk Tirokk commented Sep 11, 2023

  • In case no language is detected set default.

🍰 Pullrequest

Fix post language detection.

Issues

Todo

  • None

@Tirokk
Fix post language detection
b5ae101 
- In case no language is detected set default.
@Tirokk Tirokk requested a review from Mogge September 11, 2023 10:47
@Tirokk Tirokk self-assigned this Sep 11, 2023
@Tirokk Tirokk added bug Something isn't working service: backend labels Sep 11, 2023
@Tirokk Tirokk added this to the 🏃 The Sprint after that... milestone Sep 11, 2023
@Tirokk Tirokk requested review from mahula and Elweyn September 11, 2023 10:49
Mogge
Mogge approved these changes Sep 12, 2023
View reviewed changes
Copy link
Contributor

@Mogge Mogge left a comment

Choose a reason for hiding this comment

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

Though I do not understand how language detection can fail. How can the text be empty?

@Elweyn Elweyn merged commit 4d66e10 into master Sep 12, 2023
@Elweyn Elweyn deleted the 6712-fix-set-post-language-if-undedected branch September 12, 2023 07:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@Mogge Mogge Mogge approved these changes

@Elweyn Elweyn Elweyn approved these changes

@mahula mahula Awaiting requested review from mahula

Assignees

@Tirokk Tirokk

Labels
bug Something isn't working service: backend
Projects
Kanban Board Ocelot-Social Software
Status: Done
Milestone
🏃 The Sprint after that...
Development

Successfully merging this pull request may close these issues.

🐛 [Bug] Backend - Setting post language fails, if no language was detected
3 participants
@Tirokk @Mogge @Elweyn