Refactor AddNewTranslationContext

translation-cards / translation-cards

An app that enables refugees and field workers to communicate more effectively and confidently.

http://translation.cards/

Apache License 2.0

27 stars 20 forks source link

Refactor AddNewTranslationContext #265

Open DeuceTheCoder opened 8 years ago

DeuceTheCoder commented 8 years ago

Refactor NewDeckContext to use a builder pattern, and instantiate the NewTranslation in the Summary Activity. Remove the responsibility for saving the Translation from the Context, and instead make calls to the translation service from the containing Activity or Service.

DeuceTheCoder commented 8 years ago

Open questions:

Is it necessary to return a NewTranslation from this builder?
- can Translation and the DictionaryService replace the need for this class?