-
Notifications
You must be signed in to change notification settings - Fork 32
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.
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
Fix fallback feedback for events and pois #1811
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you 😃 It works as expected!
Can you add an entry to CHANGELOG.md?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot, works as expected 👍
Code Climate has analyzed commit 733d204 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 66.6% (50% is the threshold). This pull request will bring the total coverage in the repository to 74.2% (0.0% change). View more on Code Climate. |
79d3977
to
56c71bf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Previously, an error was returned when the feedback was about a fallback translated object. Now, the default language gets used if required. The actual language of the feedback should not get lost, though. Co-authored-by: Timo Ludwig <ludwig@integreat-app.de>
56c71bf
to
733d204
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you 👍
Short description
This pr fixes submitting feedback about a fallback page- or poi-translation. With this pr, the default language translation gets used if required.
The actual language of the feedback should not get lost, though.
Proposed changes
Resolved issues
Fixes: #1718
Pull Request Review Guidelines