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

Allow to specify MoxyReflector package and use it from another module #117

Merged
merged 2 commits into from
May 4, 2017

Conversation

ekursakov
Copy link
Contributor

@ekursakov ekursakov commented May 3, 2017

Добавил возможность выбора пакета в котором будет сгенерирован MoxyReflector (#72).
Имя пакета задается через аргументы annotation processor'а (как здесь):

kapt {
    arguments {
        arg("moxyReflectorPackage", "com.example.common")
    }
}

Если оно не указано, то используется com.arellomobile.mvp и в нем генерируется MoxyReflector, который также включает в себя данные из других MoxyReflector'ов, зарегистрированных с помощью аннотации @RegisterMoxyReflectorPackages (идея отсюда):

@RegisterMoxyReflectorPackages({"com.example.common"})
public class App extends Application {
...

@senneco senneco merged commit 5167d89 into Arello-Mobile:master May 4, 2017
@senneco
Copy link
Collaborator

senneco commented May 4, 2017

Эх, только надо было из ветки develop фичу стартовать. Жаль я поздно заметил.

@ekursakov
Copy link
Contributor Author

У меня ветка из develop была, а вот в пулреквесте по умолчанию master стоит, а я как-то это пропустил.

@ekursakov
Copy link
Contributor Author

Есть еще некоторые сомнения по поводу конфигурации, сейчас получилось что часть параметров задается через аргументы процессора, а модули регистрируются через аннотацию.
Может стоит сделать все параметрами процессора, или как-то еще по другому?

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.

2 participants