feature: Task 4 completed #4

Gribbirg · 2024-07-13T00:58:04Z

Четвёртое задание выполнено.

Сделано по ТЗ

DI

Dagger 2
Subcomponents есть в модуле app (по большей части ради галочки)
Scope есть
Поля я как только не инжектил)
ViewModel "кастыльно" прикручены, главное работает)

добавлена многомодульность и DI между модулями через зависимости компонентов

Хранение данных

Сдано в рамках прошлой дз, так что не смотри)

Gradle

основа вынесена в conventional
сделаны таски:

проверка размера (отключение - тупо не указание максимального размера)
отправка в телегу
отправка детализации apk (если не отключено через extension)

изменено имя apk
настроен GitHub Actions

Дополнительно

добавлена сборка signed приложения

Для тестов

Apk загружен через Github Actions в артефакты и телегу

Если захочется собрать самому, то необходимо в корне проекта создать файл secrets.properties и заполнить его ключами:

YANDEX_CLIENT_ID=

TELEGRAM_BOT_API=
TELEGRAM_CHAT_ID=

KEY_STORE_PATH=
KEY_STORE_PASSWORD=
KEY_ALIAS=
KEY_PASSWORD=

# Conflicts: # .gitignore # app/.gitignore # app/build.gradle.kts # gradle/libs.versions.toml

nai1ka

DI (5/5)
✅ Использовать чистый Dagger 2 в своем приложении.
✅ Использовать иерархию Subcomponents и custom Scopes.
✅ Для добавления объектов в Component использовать минимально необходимый способ
✅ Попробовать разбить проект на модули и сделать межмодульный DI с помощью Component Dependencies.

Gradle(4/4)
✅ Вынести конфигурацию сборки проекта в convention плагины.
✅ Повторить плагин, сделанный на практике.
✅ Написать еще одну Gradle Task с названием validateApkSizeFor*
✅ Сделать так, чтобы эта задача всегда запускалась перед задачей выгрузки apk файла
✅ Выходными данными задачи должна быть информация о размере apk файла
✅ Сделать extension для плагина, который позволит отключать задачу проверки размера файла и задавать границу этого размера. Тот самый N мб.
✅ Поменять название у отправляемого apk файла.
✅ Настроить Github Actions для сборки проекта и выгрузки apk файла в телеграм на каждый ПР и линковки apk файла к CI сборке.

Очень много всего сделал, классная работа)
@nai1ka

nai1ka · 2024-07-15T09:33:34Z

app/src/main/java/ru/gribbirg/todoapp/di/modules/ListFeatureModule.kt

+    companion object {
+        @Provides
+        fun listFeatureFactory(depsImpl: AppComponent): ListFeatureFactory =
+            ListFeatureFactory(depsImpl)


Как будто здесь можно было обойтись просто @Inject в конструкторе

Суть подхода как у меня в том, чтобы сделать модули независимыми друг от друга в плане DI. Т.е. если из ListFeature DI уйдёт или переедет с Daggerа, то при этом не придётся менять остальные модули. Если делать через @Inject, то при изменении DI у ListFeature придётся менять и все модули, где он использовался, т.к. @Injectа может уже не быть(

Да и кода не сильно меньше станет, ибо придётся добавить @Binds для того, чтобы давать интерфейс ListFeatureDependencies) Но я понимаю, что под капотом @Inject и @Binds лучше одного @Provides, но в данном случае всё ради независимости модулей)

nai1ka · 2024-07-15T09:36:02Z

app/src/main/java/ru/gribbirg/todoapp/navigation/NavGraph.kt

    val animationDuration = AppTheme.dimensions.animationDurationNavigationTransition
+
+    val listViewModel = remember {
+        appComponent.listFeatureComponent().listViewModel()


Лучше переделать как в гайде. Не уверен, что это надежный способ

AlexeyZakis

DI: 5/5
Gradle: 4/4

Крутая работа!

AlexeyZakis · 2024-07-15T12:34:33Z

build-logic/src/main/kotlin/ru/gribbirg/todoapp/tasks/SizeCheckTask.kt

+        val size = file.length() / 1024
+        if (size > maxSizeKb) {
+            val text =
+                "Apk is too large!\nName: ${file.name}\nSize: $size KB\nMax size: $maxSizeKb KB"


KB можно вынести в константы. Можно забыть где-то поменять, если решишь отображать размер, например, в МБ

AlexeyZakis · 2024-07-15T12:34:58Z

build-logic/src/main/kotlin/ru/gribbirg/todoapp/tasks/SizeCheckTask.kt

+
+        val file = apkDir.get().findApk()
+
+        val size = file.length() / 1024


1024 - магическое число

Uncreated

В целом все ок)

Uncreated · 2024-07-16T16:18:03Z

app/src/main/java/ru/gribbirg/todoapp/TodoApplication.kt

-            internetDataStore = internetDataStore,
-            listsMerger = listMergerImpl,
-        )
+    internal val appComponent by lazy {


Можно lazy(NONE), т.к. не нужна синхронизация на мейн треде

Uncreated · 2024-07-17T10:39:00Z

app/src/main/java/ru/gribbirg/todoapp/navigation/NavGraph.kt

+    }
+
+    val editViewModelFactory = remember {
+        appComponent.editFeatureComponent().editFeatureFactory().createViewModelFactory()


Пустая строка

Uncreated · 2024-07-17T10:39:07Z

app/src/main/java/ru/gribbirg/todoapp/navigation/NavGraph.kt

@@ -44,13 +51,13 @@ fun NavGraph(
                )
            },
        ) {
-            TodoListItemScreen(
+            ru.gribbirg.list.TodoListItemScreen(


Лучше импортнуть и не писать весь путь

Uncreated · 2024-07-17T10:39:38Z

app/src/main/java/ru/gribbirg/todoapp/navigation/NavGraph.kt

                viewModel = listViewModel,
                toEditItemScreen = { id ->
                    navController.navigate(Screen.Edit.getRoute(itemId = id)) {
                        launchSingleTop = true
                    }
-                }
+                },


Запятые, конечно, хорошо, если есть понимание зачем они пишутся в конце)

Uncreated · 2024-07-17T10:46:27Z

feature/list/src/main/java/ru/gribbirg/list/di/ListFeatureFactory.kt

+
+import ru.gribbirg.list.TodoItemsListViewModel
+
+class ListFeatureFactory(deps: ListFeatureDependencies) {


Класс лишний, можно все тоже самое сделать напрямую. ListFeatureDependencies можно получать напрямую из AppComponent как зависимость. Сам класс зависимостей должен храниться в модуле рядом с компонентом ListFeatureComponent, т.о. все равно кто будет реализовывать эти зависимости

Gribbirg added 30 commits July 8, 2024 17:50

feat: conventional plugin

f733af9

feat: telegram reporter

1a53bdf

fix: git hub actions push add

bec85ac

fix: git keys fix

7d3af05

fix: githun actions debug

650f9d9

fix: github actions env var

a4fac2d

fix: github actions env var v2

ac13606

fix: github actions debug try

fcd77bb

fix: github actions secrets fix

dd3138c

fix: github secrets fix

ae07cc6

fix: github actions decryption

1b83fad

fix: github actions decryption change

88e31a0

fix: github actions decryption change tru 2

78286dd

fix: github actions decryption change try 3

474faf7

fix: github actions decryption change try 4

bc9fc2f

fix(github actions): keystore change

32a2f26

fix(github actions): final

4560574

Merge branch 'refs/heads/master' into fourth-task

6656d72

# Conflicts: # .gitignore # app/.gitignore # app/build.gradle.kts # gradle/libs.versions.toml

feat: merge branches, apk renaming

4118cf1

fix: apk naming on github actions

9be4c5f

fix: apk find

ba1a010

build: added check of size of apk

167c3a6

fix(reporter plugin): output

ffd2bde

fix: compile apk version update

6f8c7bc

build: report with details plugin

e0f73c2

fix: github actions apk upload

7e7b258

fix: github actions apk upload try 2

c790154

fix: github actions apk upload try 4

b4978f6

fix: github actions apk upload try 5

501acd8

fix: github actions apk upload try 6

6ac304b

Gribbirg added 9 commits July 9, 2024 23:42

fix: github actions apk upload try 7

d71a617

refactor: architecture clean up

9fa12c8

refactor: system data provider move to self-generated key

70ff018

refactor(di): dagger added

c524569

refactor: viewmodels add to di

b8d2738

refactor: multi model

d879d56

refactor: subcomponents and scopes setup

c03a66b

refactor: conventions plugins

d0f927f

refactor: conventions plugins

b74c7cb

nai1ka reviewed Jul 15, 2024

View reviewed changes

AlexeyZakis suggested changes Jul 15, 2024

View reviewed changes

Gribbirg merged commit 92abea9 into master Jul 15, 2024
1 check passed

Uncreated reviewed Jul 17, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feature: Task 4 completed #4

feature: Task 4 completed #4

Gribbirg commented Jul 13, 2024 •

edited

Loading

nai1ka left a comment

nai1ka Jul 15, 2024

Gribbirg Jul 15, 2024 •

edited

Loading

nai1ka Jul 15, 2024

AlexeyZakis left a comment

AlexeyZakis Jul 15, 2024

AlexeyZakis Jul 15, 2024

Uncreated left a comment

Uncreated Jul 16, 2024

Uncreated Jul 17, 2024

Uncreated Jul 17, 2024

Uncreated Jul 17, 2024

Uncreated Jul 17, 2024


		val file = apkDir.get().findApk()

		val size = file.length() / 1024


		import ru.gribbirg.list.TodoItemsListViewModel

		class ListFeatureFactory(deps: ListFeatureDependencies) {

feature: Task 4 completed #4

feature: Task 4 completed #4

Conversation

Gribbirg commented Jul 13, 2024 • edited Loading

Сделано по ТЗ

DI

Хранение данных

Gradle

Дополнительно

Для тестов

nai1ka left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Gribbirg Jul 15, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

AlexeyZakis left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Uncreated left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Gribbirg commented Jul 13, 2024 •

edited

Loading

Gribbirg Jul 15, 2024 •

edited

Loading