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.

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

[Backport] Make collaborative legislation translatable #2912

Merged
merged 10 commits into from
Sep 24, 2018

Conversation

javierm
Copy link
Member

@javierm javierm commented Sep 19, 2018

References

Objectives

Quoting from AyuntamientoMadrid#1628 (all credit to Marko):

Allow Admin Notifications to exist in multiple languages. This applies to 2 of its fields: title and body. Also a couple of fixes.

Notes

  • Proposal categories aren't translated as part of this PR. There's another issue for that: consul#2874.
  • As with the original Milestones PR, data migration is not implemented.

mlovic and others added 10 commits September 19, 2018 14:54
Due to its markdown editor, the admin draft version page
requires JS to correctly render the draft version form.
They were accidentally changed from `text_area` to `text_field` when
making them translatable.
There's no reason to only convert Markdown to HTML in translations when
their body changes but to always convert it when the "main" body field
changes.

Whether we should always use the condition or never use it is something
we can debate about, though.
We were expecting translation parameters in legislation processes
`update` action. However, those parameters aren't sent when we get to
that action through the "proposals" tab.
@javierm javierm merged commit ef0192c into master Sep 24, 2018
@javierm javierm deleted the backport-make-collaborative-legislation-translatable branch September 24, 2018 15:54
@decabeza decabeza added this to Release 0.17 in Roadmap Oct 20, 2018
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.

None yet

3 participants