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.

Sign up for GitHub

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

Jump to bottom

154 add safe args #159

Merged
merged 54 commits into from
Oct 21, 2024
Merged

154 add safe args #159

merged 54 commits into from
Oct 21, 2024

Conversation

Eka666Sed
Copy link
Contributor

@Eka666Sed Eka666Sed commented Aug 24, 2024 •
edited
Loading

  • подключает плагин safeArgs

  • подключает плагин parcelize

  • заменяет передачу аргументов между экранами через safeArgs

  • редактирует навигацию между фрагментами

  • использует аргументы как объекты, где это возможно

@Eka666Sed Eka666Sed requested a review from a team August 24, 2024 12:18
@Eka666Sed Eka666Sed self-assigned this Aug 24, 2024
@Eka666Sed Eka666Sed requested review from muskilon and DimaK21 and removed request for a team August 24, 2024 12:18
@Eka666Sed Eka666Sed requested a review from Secur1ty-21 as a code owner August 24, 2024 12:18
@Eka666Sed Eka666Sed linked an issue Aug 24, 2024 that may be closed by this pull request
Подключить safeArgs #154
Open
Eka666Sed added 11 commits August 26, 2024 14:32
@Eka666Sed
154_фиксит баг перехода SelectMovieFragment на MatchedSessionListFrag…
52795e4 
…ment
@Eka666Sed
154_фиксит очистку backstack до MainFragment
b9fe737
@Eka666Sed
154_редактирует OnboardingDataProvider и переходы на экраны после онб…
3f46b98 
…ордингов
@Eka666Sed
154_редактирует навинацию
8df0bb1
@Eka666Sed
154_убирает лишний фрагмент на онбордингах
a521ef6
@Eka666Sed
154_убирает закоммиченный код
5ca5472
@Eka666Sed
154_заменяет получение объекта onboardingItem напрямую
1087942
@Eka666Sed
154_заменяет получение объекта onboardingItem напрямую в OnboardingFr…
b044503 
…agment
@Eka666Sed
154_использует @parcelize и Safe Args для movieDetails
3a261c4
@Eka666Sed
154_редактирует остальные фрагменты, после замены movieDetails на объект
9ed22cb
@Eka666Sed
154_завершает переход на SafeArgs с использованием объектов, где это …
7ec5565 
…было возможно
DimaK21
DimaK21 approved these changes Aug 29, 2024
View reviewed changes
Copy link
Contributor

@DimaK21 DimaK21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

закомменченное назад только вернуть не забыть

Secur1ty-21
Secur1ty-21 reviewed Sep 1, 2024
View reviewed changes
Copy link
Contributor

@Secur1ty-21 Secur1ty-21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличная работа!

@Eka666Sed
Merge remote-tracking branch 'origin/develop' into 154_add_safeArgs
8c7074e 
# Conflicts:
#	app/src/main/java/com/davay/android/feature/registration/presentation/RegistrationFragment.kt
#	app/src/main/java/com/davay/android/feature/splash/presentation/SplashFragment.kt
Eka666Sed and others added 17 commits September 2, 2024 20:18
@Eka666Sed
154_убирает nullable переменную OnboardingItem, переносит args в обла…
2e9bb72 
…сть полей класса в OnboardingFirstFragment
@Eka666Sed
154_убирает неиспользуемый входной аргумент в ChangeNameBottomSheetFr…
13b3ec3 
…agment
@Eka666Sed
154_убирает неиспользуемый входной аргумент в MatchBottomSheetFragment
8fe29ee
@Eka666Sed
154_убирает замечания detekt в SelectMovieFragment
522d6a7
@Eka666Sed
154_убирает код, создающий сессию вручную
b3f8ff8
154_изменение навигации для ChangeNameBottomSheetFragment через navGraph
6a5fdfd
@Eka666Sed
154_переносит навигацию через navGraph, вместо newInstance в MatchBot…
09f0152 
…tomSheetFragment
@Eka666Sed
154_устраняет bug передачи аргументов
da6755b
@Eka666Sed
154_устраняет bug отсутствия анимации в MatchBottomSheetFragment
a22652a
@Eka666Sed
Merge remote-tracking branch 'origin/develop' into 154_add_safeArgs
3f32284 
# Conflicts:
#	app/src/main/java/com/davay/android/feature/matchedsession/presentation/MatchedSessionFragment.kt
@Eka666Sed
154_вливает develop, решает конфликты
34513d7
@Eka666Sed
154_редактирует class MatchBottomSheetArgs
734e8f6
@Eka666Sed
154_возвращает action, убирает Deprecated и пустую функцию
a18983e
@Eka666Sed
154_редактирует метод subscribe, чтобы передавать session через args.…
b87ee3f 
…session напрямую
@Eka666Sed
154_возвращает метод с incrementAnimation и переписывает его на SafeArgs
b6e4446
@Eka666Sed
Merge remote-tracking branch 'origin/develop' into 154_add_safeArgs
72ac93c 
# Conflicts:
#	app/src/main/java/com/davay/android/core/domain/models/Session.kt
#	app/src/main/java/com/davay/android/feature/createsession/presentation/createsession/CreateSessionFragment.kt
@Eka666Sed
154_передает аргументы между CreateSession и WaitSession через SafeArgs
fdc53a5
Eka666Sed and others added 10 commits September 18, 2024 10:57
@Eka666Sed
154_удаляет излишний вызов метода subscribe()
422d0de
@Eka666Sed
154_делает поле movieDetails не nullable
b9238e7
@Eka666Sed
154_убирает проверку movieDetails на nullable во фрагменте
be53923
@Eka666Sed
154_возвращает комментарий
e6e26de
@Eka666Sed
154_убирает внешнюю nullable переменную session
9f14d9f
@Eka666Sed
Merge remote-tracking branch 'origin/develop' into 154_add_safeArgs
d8547b0 
# Conflicts:
#	app/src/main/java/com/davay/android/base/BaseViewModel.kt
#	app/src/main/java/com/davay/android/feature/coincidences/presentation/CoincidencesFragment.kt
#	app/src/main/java/com/davay/android/feature/main/presentation/MainFragment.kt
#	app/src/main/java/com/davay/android/feature/matchedsession/presentation/MatchedSessionFragment.kt
#	app/src/main/java/com/davay/android/feature/onboarding/presentation/OnboardingFragment.kt
#	app/src/main/java/com/davay/android/feature/registration/presentation/RegistrationFragment.kt
#	app/src/main/java/com/davay/android/feature/sessionlist/presentation/SessionListFragment.kt
#	app/src/main/java/com/davay/android/feature/sessionsmatched/presentation/MatchedSessionListFragment.kt
#	app/src/main/java/com/davay/android/feature/sessionsmatched/presentation/adapters/SessionListAdapter.kt
#	app/src/main/java/com/davay/android/feature/waitsession/presentation/WaitSessionFragment.kt
@Eka666Sed
154_вливает develop
e8917a0
@Eka666Sed
154_удаляет закоммиченный код
2b51dcd
154_Move comment
7e10dfe
Merge branch 'develop' into 154_add_safeArgs
a2ccef8 
# Conflicts:
#	app/src/main/java/com/davay/android/feature/createsession/presentation/createsession/CreateSessionViewModel.kt
#	app/src/main/java/com/davay/android/feature/main/presentation/MainFragment.kt
#	app/src/main/java/com/davay/android/feature/selectmovie/presentation/SelectMovieFragment.kt
#	app/src/main/java/com/davay/android/feature/waitsession/presentation/WaitSessionFragment.kt
#	app/src/main/java/com/davay/android/feature/waitsession/presentation/WaitSessionViewModel.kt
@Secur1ty-21 Secur1ty-21 merged commit e1acf24 into develop Oct 21, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Secur1ty-21 Secur1ty-21 Secur1ty-21 approved these changes

@muskilon muskilon muskilon approved these changes

@DimaK21 DimaK21 DimaK21 approved these changes

Assignees

@Eka666Sed Eka666Sed

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Подключить safeArgs
4 participants
@Eka666Sed @Secur1ty-21 @muskilon @DimaK21