-
Notifications
You must be signed in to change notification settings - Fork 1
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
Internal evaluations export improvements (from 0.29-stable) #12
base: main
Are you sure you want to change the base?
Internal evaluations export improvements (from 0.29-stable) #12
Conversation
Added the new "proposal_id" in the export, it's available in staging already: https://decidim-lot2.populate.tools/admin/participatory_processes/branch-highway/components/4/manage/proposals?q%5Bs%5D=translated_title+asc Regarding the missing proposal titles and descriptions, I can confirm that when the translation is not available, we don't fallback to the default locale in those exports. I believe this is something that happens in all exports. Also, I tried exporting to Excel and worked now. 🙌 |
Great! It works perfect :) Thanks @mllocs |
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.
Why are you doing the PR against the release/0.29-stable
branch. Is it because these changes don't make sense in decidim >0.29?
If that's not the case, I would do this PR against main
and then backport it to release/0.29-stable
and release/0.28-stable
if needed.
328d6de
to
b88c2f5
Compare
@fblupi It was mostly for convenience, this way it was easier to merge into our staging which uses [email protected]. I rebased the PR and pointed it to |
Closes #9