Skip to content

Identifying translatable fields in meetings and comments#6333

Merged
tramuntanal merged 3 commits intodecidim:feat/machine-translationfrom
codegram:identify_translatable_fields_for_meetings
Jul 29, 2020
Merged

Identifying translatable fields in meetings and comments#6333
tramuntanal merged 3 commits intodecidim:feat/machine-translationfrom
codegram:identify_translatable_fields_for_meetings

Conversation

@anaghavl
Copy link
Copy Markdown
Contributor

@anaghavl anaghavl commented Jul 23, 2020

As part of #6127,
Applied TranslatableResource to meetings and comments.

@anaghavl anaghavl requested a review from mrcasals July 23, 2020 15:13
@andreslucena andreslucena changed the title Idenifying translatable fields in meetings and comments Identifying translatable fields in meetings and comments Jul 27, 2020
@anaghavl anaghavl force-pushed the feat/machine-translation branch 2 times, most recently from 4ae9f10 to 5cbf1c3 Compare July 27, 2020 12:09
@tramuntanal
Copy link
Copy Markdown
Contributor

Hi @anaghavl can you link in the description from where this PR comes from please?
e.g. the typical "related to" field.

By the way, you have conflicts, thanks for checking 😃

@anaghavl anaghavl force-pushed the identify_translatable_fields_for_meetings branch from ffea7d8 to 6d8bb5e Compare July 28, 2020 04:57
@anaghavl
Copy link
Copy Markdown
Contributor Author

Hi @anaghavl can you link in the description from where this PR comes from please?
e.g. the typical "related to" field.

By the way, you have conflicts, thanks for checking 😃

Sorry, done and done 😅

@mrcasals
Copy link
Copy Markdown
Contributor

@tramuntanal @Leusev all green now, can you re-review it please? 😄

@anaghavl anaghavl force-pushed the feat/machine-translation branch from 5cbf1c3 to 6cd43aa Compare July 28, 2020 11:36
@anaghavl anaghavl force-pushed the identify_translatable_fields_for_meetings branch from 6d8bb5e to db00c28 Compare July 28, 2020 12:04
@tramuntanal
Copy link
Copy Markdown
Contributor

I've re-run tests for 6 workflows that got stuck. Take into account that there are problems with GitHub Actions https://github.community/t/jobs-randomly-hanging-at-starting-your-workflow-run/124295/7, we must supervise that all tests execute in our PR and re-run the hanging ones until github solves it

@mrcasals
Copy link
Copy Markdown
Contributor

@tramuntanal thanks for taking care of this! It should be all green though, can you review it please? 😄

@mrcasals
Copy link
Copy Markdown
Contributor

@tramuntanal all green now! 😄

@tramuntanal tramuntanal merged commit e645b9d into decidim:feat/machine-translation Jul 29, 2020
@mrcasals mrcasals deleted the identify_translatable_fields_for_meetings branch July 29, 2020 07:28
mrcasals added a commit that referenced this pull request Aug 14, 2020
* Base branch

* remove file

* Idenifying translatable fields in meetings and comments

Co-authored-by: Marc Riera Casals <mrc2407@gmail.com>
mrcasals added a commit that referenced this pull request Aug 14, 2020
* Base branch

* remove file

* Idenifying translatable fields in meetings and comments

Co-authored-by: Marc Riera Casals <mrc2407@gmail.com>
tramuntanal pushed a commit that referenced this pull request Aug 17, 2020
* Base branch

* remove file

* Base branch

* remove file

* Identify translatable resources (#6145)

* Base branch

* remove file

* Require confirmation on exiting a survey mid-answering (#6118)

* Require confirmation on exit

* Add specs

* Use path instead of url

* Fix changelog

* Trigger build

* Fix expected path on test

* Fix method call

* Take textareas and selects into account

* WIP adding concern

* Adding concern in all the models which have translatable fields

* removed :extended_data as translatable field

* WIP adding concern

* Adding concern in all the models which have translatable fields

* Revert "Require confirmation on exiting a survey mid-answering (#6118)"

This reverts commit bdeb933.

* Revert "remove file"

This reverts commit 2565dbb.

* Revert "Base branch"

This reverts commit 2a09cc4.

Co-authored-by: Marc Riera Casals <mrc2407@gmail.com>

* Add Decidim global and organization config for machine translation (#6128)

* Adding setting to organizations table and creating global config

* Adding config accessor to core.rb

* Base branch

* Added a check to display machine translation settings and changed initializer value

* Fixing lint issue in migration file

* Adding test and removing test file

Co-authored-by: decidim-bot <decidim-bot@users.noreply.github.com>
Co-authored-by: anagha <anagha1996@gmail.com>

* Identifying translatable fields in meetings and comments (#6333)

* Base branch

* remove file

* Idenifying translatable fields in meetings and comments

Co-authored-by: Marc Riera Casals <mrc2407@gmail.com>

* Identifying translatable fields for proposals (#6346)

* Add machine translation service (#6179)

Co-authored-by: Marc Riera Casals <mrc2407@gmail.com>
Co-authored-by: Marc Riera <mrc2407@gmail.com>

* Make some fields non-translatable

* Improve spec

* Don't run job if class is not defined

* Improvee method naming

* Machine translation display priority (#6385)

* Add docs on how to enable the integration

* Add docs on how to write a machine translation service

* Improve code strength

* Fix specs

* Fix specs

Co-authored-by: anagha vl <44900292+anaghavl@users.noreply.github.com>
Co-authored-by: decidim-bot <decidim-bot@users.noreply.github.com>
Co-authored-by: anagha <anagha1996@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants