Skip to content

Add Conferences and Admin missing translations#7653

Merged
mrcasals merged 14 commits intodecidim:developfrom
CodiTramuntana:conference_missing_transations
Mar 26, 2021
Merged

Add Conferences and Admin missing translations#7653
mrcasals merged 14 commits intodecidim:developfrom
CodiTramuntana:conference_missing_transations

Conversation

@tramuntanal
Copy link
Copy Markdown
Contributor

@tramuntanal tramuntanal commented Mar 19, 2021

🎩 What? Why?

The admin form for conferences has some untranslated fields.
This PR adds these fields to the corresponding locales file.

As a bonus, two fixes in conferences_invites/index.html.erb and conferences_registrations/index.html.erb.

📌 Related Issues

Link your PR to an issue

  • Fixes #?

Testing

Describe the best way to test or validate your PR.
As an admin, when I'm editing the conference information, I expect all fields to be translated to my current locale.
Some of the untranslated fields are: Objectives, Location, Start date, End date, Registrations enabled, ...

📋 Checklist

🚨 Please review the guidelines for contributing to this repository.

  • CONSIDER adding a unit test if your PR resolves an issue.
  • ✔️ DO check open PR's to avoid duplicates.
  • ✔️ DO keep pull requests small so they can be easily reviewed.
  • ✔️ DO build locally before pushing.
  • ✔️ DO make sure tests pass.
  • ✔️ DO make sure any new changes are documented in docs/.
  • ✔️ DO add and modify seeds if necessary.
  • ✔️ DO add CHANGELOG upgrade notes if required.
  • ✔️ DO add to GraphQL API if there are new public fields.
  • ✔️ DO add link to MetaDecidim if it's a new feature.
  • AVOID breaking the continuous integration build.
  • AVOID making significant changes to the overall architecture.

📷 Screenshots

The admin form for Conferences with some of the fields out of translations
image

The admin form for Organization with admin_terms_of_use_body field out of translations
imatge

♥️ Thank you!

@tramuntanal tramuntanal marked this pull request as draft March 19, 2021 15:46
@mrcasals
Copy link
Copy Markdown
Contributor

@tramuntanal thanks for the PR! IGnoring the initiatives failures (it's a flaky and it's being solved), the job for the main module complains about i18n keys not being normalized. Could you review that, please?

@tramuntanal tramuntanal marked this pull request as ready for review March 22, 2021 17:02
body: Cos
subject: Assumpte
organization:
admin_terms_of_use_body: Body for the admin terms of use
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@tramuntanal this shouldn't modify the ca.ymlfile, but theen.yml`!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

OMG! yes thanks!

@tramuntanal
Copy link
Copy Markdown
Contributor Author

Change applied

@mrcasals mrcasals changed the title Add Conferences missing translations Add Conferences and Admin missing translations Mar 26, 2021
@mrcasals mrcasals added module: admin type: fix PRs that implement a fix for a bug module: conferences labels Mar 26, 2021
@mrcasals mrcasals merged commit 39c7f7f into decidim:develop Mar 26, 2021
@tramuntanal tramuntanal deleted the conference_missing_transations branch March 29, 2021 06:13
tramuntanal added a commit to CodiTramuntana/decidim that referenced this pull request Mar 29, 2021
tramuntanal added a commit to CodiTramuntana/decidim that referenced this pull request Mar 29, 2021
entantoencuanto added a commit that referenced this pull request Mar 31, 2021
* develop: (26 commits)
  Fix trustees admin menu (#7772)
  Do not modify the controller class in the controller tests that render views (#7755)
  Add HTML escaping to the expectations as the strings are escaped (#7760)
  Add automated accessibility audit + HTML validation to CI pipeline (#7751)
  fix(elections): js assets manifest (#7759)
  Add admin missing translations (#7702)
  Add Conferences and Admin missing translations (#7653)
  New Crowdin updates (#7735)
  Improve vote flow (#7682)
  Strip the <p> tags from inside the heading elements (#7732)
  Fix the date cell spec failing randomly close to day changes (#7703)
  Change the timeline date color for accessible color contrast against its background (#7750)
  Remove the opacity from process upcoming/past/all filters for accessible contrast (#7749)
  Fix color contrast against the sidebar navigation background (#7748)
  Validate the HTML for the account page (#7747)
  Fix report modal form accessibility (#7746)
  Accessibility fixes for conversations (#7745)
  Add a landmark ARIA role to the cookie banner (#7738)
  Fix HTML validation on standalone content page (#7744)
  Add aria-label to the area filter on participatory space pages (#7743)
  ...
tramuntanal added a commit to CodiTramuntana/decidim that referenced this pull request Apr 7, 2021
…issing_transations

Add Conferences and Admin missing translations (decidim#7653)
laurajaime pushed a commit to CodiTramuntana/decidim that referenced this pull request May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: admin module: conferences type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants