Introduce the Part IV (Linus Torvalds) solution #4
Conversation
There was a problem hiding this comment.
Конкретно по 4-й домашке все ок, правда сложно конечно проверить Firebase вещи без доступа к твоему аккаунту)
Единственное - флейворы сделаны через dart-define и они не ведут себя, как обычные флейворы (например, они должны ставиться как 2 разных приложения). Но я сам тоже решил через dart-define сделать, а потом заметил такие вещи.
Также имеется странное поведение приложения, не совсем понял с чем это связано (см. видео).
tasky_bugs.mp4
И еще когда всего лишь одна задача, немного странно выглядит (см. скриншот).
android/app/google-services.json
Outdated
There was a problem hiding this comment.
Вроде как google-services.json, firebase.json, firebase_options.dart и т.п. лучше не добавлять в git, т.к. там есть секретные данные. А для билда в workflows надо эту информацию добавить в секреты, как ты добавил токены и информацию по Firebase App Distribution. Но я на самом деле тоже уже это все спалил в гитхабе, а потом только ментор сказал, что лучше сделать так :D
There was a problem hiding this comment.
Я опирался на этот ответ, где с одобрения разработчиков указывается, что коммитить эти файлы в репозиторий — это норм
README.md
Outdated
|
|
||
| ## APK | ||
| [Загрузить файл .apk](https://github.com/eshfield/tasky/releases) | ||
| Загрузить .apk файл можно в разделе [Releases](https://github.com/eshfield/tasky/releases) или забрать у последнего запуска в [GitHub Actions](https://github.com/eshfield/tasky/actions/) | ||
|
|
||
| ## Локальный запуск | ||
| Для корректной работы при локальном запуске необходимо указать токен авторизации: | ||
| ``` | ||
| --dart-define=TOKEN=%YOUR_TOKEN% |
There was a problem hiding this comment.
--dart-define=TASKY_API_TOKEN=%YOUR_TOKEN%
There was a problem hiding this comment.
Справедливо, осталось от старой версии
There was a problem hiding this comment.
Круто) я не успел сделал прослушку стрима
lib/core/extensions/hex_color.dart
Outdated
There was a problem hiding this comment.
Выглядит так, что можно подать строку, не содержащую цвет, и это место упадет.
There was a problem hiding this comment.
Хорошее замечание, стоит добавить проверку
Я решил не загромождать Readme скриншотами Firebase аккаунта, так как корректность интеграции в принципе можно оценить по коду — в самом Firebase аккаунте сложно что-то не так сделать
Как я понял, авторы задания изначально симпатизируют решению через
Вот это лихо приложение расколбасило — попробую воспроизвести что тут могло случиться, сам с таким не встречался когда тестировал
Это хорошее наблюдение. Landscape-ориентацию стоит включать когда есть больше одной задачи |
Fourth part of the Todo List App task