Feature/170 сonfigured app distribution #174
Conversation
Made: 1. api 2. response 3. dto 4. DataMapper 5. Exception mapper
Made: 1. di 2. Repository 3. Interactor 4. DomainMapper 5. Data 6. State 7.ExceptionToErrorMapper
Made: 1. PasswordRecoveryViewModel 2. State
…m/CashAdv/CashADV into 71-Password-Recovery
Made:
1. Solved the "empty message exception" 2. Renamed the class names to more understandable ones
1. Resolved conflicts
1. Made the lines shorter
1) removed context from viewmodel 2) added Resources in viewmodel 3) renamed fun setEmail(email:String) on requestRecovery() 4) removed isResetPasswordInProgress() 5)I removed the extra one logDebugMessage 6)formatted the code 7) removed DTO from Domain 8) the code field is cleared when an error occurs
1) Called onBackPressedDispatcher.addCallback in viewmodel 2) added a theme
1) made the creation of a unique version number 2) added notification to testers about new builds
1) I did that triggers the workflow not only for dev branch 2) Did that buildNumber = GITHUB_SHA::7
MajoritySky2496
requested review from
alexxk2,
Barsec13,
avanisimov and
artwist-polyakov
| @@ -0,0 +1,128 @@ | |||
| package app.cashadvisor.authorization.data | |||
There was a problem hiding this comment.
это из другого пулл-реквеста, надо убрать
There was a problem hiding this comment.
удалить эти файлы и сделать новый push ?
There was a problem hiding this comment.
лучше сделать чистую ветку от dev и там модифицировать только то, что нужно для задачи
There was a problem hiding this comment.
сделай чистый PR и тогда можно будет нормально смотреть что ты сделал
| pull_request: | ||
| types: | ||
| - closed | ||
| branches: [ "dev" ] | ||
|
|
There was a problem hiding this comment.
а почему убрать проверку на dev-бранч?
There was a problem hiding this comment.
я так понял уведомления будут отправляется если только сделать мердж ветки dev в main, но нам же для всех веток надо? Поэтому решил убрать dev.
There was a problem hiding this comment.
это условие будет срабатывать при закрытии PR в ветку dev
то есть когда мы мерджим PR в dev, тут всё правильно
No description provided.