🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 #9

changhwan77 · 2024-02-16T14:27:17Z

Title

🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가

What type of PR is this?

Description

유저가 마이페이지 탭에서 로그아웃 버튼을 클릭 시 트티 홈 탭으로 이동시키는 로직 추가

As-is: 마이페이지 탭에서 로그아웃 버튼을 클릭해도 마이페이지에 유저가 남아있었음.
To-Be: 위 문제 해결

회원가입 > 개인 유저 > 전체 동의 버튼 관련 오류 수정

As-is: 전체 동의를 클릭 후 전체 동의 버튼 이외에 다른 버튼을 누르면 모든 버튼의 체크가 사라짐
To-Be: 위 문제 해결 + 전체 동의 버튼 이외에 다른 버튼이 모든 클릭 되어도 전체 동의 버튼의 체크 표시가 안되도록 변경

트티 홈 메인 배너 추가 및 스크롤 시 숨겨지도록 기능 추가

As-is: ListView에 메인 배너와 GridView를 사용할 시 배너가 스크롤이 안되고 GridView가 높이를 무한으로 가지려고 해 Layout crush 발생.
To-Be: CustomScrollView 사용하여 위 문제를 해결

Mobile & Desktop Screenshots/Recordings

default.mp4

Checklist:

My code follows the style guidelines of this project
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
Any dependent changes have been merged and published in downstream modules

To Reviewers

build 메서드에서 토큰 인증 등을 체크해도 되는지 궁금합니다..!

changhwan77 · 2024-02-16T14:29:09Z

lib/common/main_navigation_screen.dart

-  late int _selectedIndex = _tabs.indexOf(widget.tab);
+  @override
+  Widget build(BuildContext context) {
+    int selectedIndex = ref.watch(navigationSelectedIndexProvider);


마이페이지 탭에서 로그아웃 시 selectedIndex를 넘겨주어야 홈 탭으로 이동할 수 있기 때문에 위젯 간 해당 데이터를 공유하기 위해 상태 관리 솔루션을 사용했어요.

changhwan77 · 2024-02-16T14:30:38Z

lib/common/main_navigation_screen.dart

+      setState(() {
+        // 유저가 클릭한 index를 navigationSelectedIndexProvider의 state에 할당
+        ref.read(navigationSelectedIndexProvider.notifier).state = index;
+      });


onTap 에서 유저가 클릭한 index를 받아 navigationSelectedIndexProvider가 관리하고 있는 selectedIndex 값을 업데이트 해주면서 setState함수를 호출해 화면을 다시 그려요 -> 유저가 클릭한 탭의 화면으로 이동

changhwan77 · 2024-02-16T14:32:04Z

lib/common/service/navigation_index_provder.dart

+    'profile',
+  ];
+
+  final tab = ref.watch(tabsProvider);


router.dart에서 MainNavigationScreen으로 넘기는 tab 값을 tabsProvider로 관리해요.

changhwan77 · 2024-02-16T14:33:28Z

lib/common/service/navigation_index_provder.dart

+
+  final tab = ref.watch(tabsProvider);
+
+  int selectedIndex = tabs.indexOf(tab);


그렇게 받은 tab 값을 tabs 라는 리스트를 순회하면서 같은 값을 찾으면 리스트에서 해당 값이 위치한 곳(index)를 selectedIndex로 앱 실행 후 초기 값을 할당해요.

changhwan77 · 2024-02-16T14:33:57Z

lib/common/service/tab_provider.dart

@@ -0,0 +1,3 @@
+import 'package:flutter_riverpod/flutter_riverpod.dart';
+
+final tabsProvider = StateProvider<String>((ref) => 'tuti');


앱 실행 시 화면 진입점은 tuti 화면이기 때문에 초기 값을 tuti로 설정해줬어요.

changhwan77 · 2024-02-16T14:45:17Z

lib/features/auth/views/join_private_screen.dart

+                        style: Theme.of(context)
+                            .textTheme
+                            .bodyMedium!
+                            .copyWith(fontWeight: FontWeight.w600),
                        controller: _nameController,
                        onEditingComplete: () {
                          if (_name.isNotEmpty) {


TextField의 폰트가 너무 커서 textTheme의 bodyMedium으로 통일

changhwan77 · 2024-02-16T14:46:54Z

lib/features/profile/views/profile_screen.dart

-                            });
+                            ref
+                                .read(navigationSelectedIndexProvider.notifier)
+                                .state = 1;


유저가 로그아웃 버튼을 클릭 시 토큰을 제거하고 navigationSelectedIndexProvider의 상태를 1(tuti 탭 index)로 변경했어요

-> mainNavigationScreen의 build 내 watch로 청취중인 navigationSelectedIndexProvider가 변경을 감지하고 reBuild하여 트티 탭으로 이동하게 수정했어요.

changhwan77 · 2024-02-16T14:47:50Z

lib/features/tutis/widgets/tuti_widgets/main_banner.dart

트티 홈 화면 메인 배너 추가 및 클릭 시 퍼스널 브랜딩 화면으로 이동하도록 설정

changhwan77 · 2024-02-16T14:52:13Z

lib/features/tutis/widgets/tuti_widgets/tuti_card_mobile.dart

+            flexibleSpace: const FlexibleSpaceBar(
+              background: TuTiBanner(),
+            ),
+          ),


95 ~ 100번째 코드 설명

유저가 스크롤 시 메인 배너를 숨기도록 하기 위해 CustomScrollView 사용했어요.

SliverAppBar의 pinned(스크롤 해도 상단에 고정), floating(유저가 스크롤을 위로 올릴 시 바로 메인 배너가 튀어나옴) 을 false로 설정했어요.

changhwan77 · 2024-02-16T14:54:09Z

lib/router.dart

-          return MainNavigationScreen(tab: tab);
+          String tab = state.params['tab'] ?? 'tuti';
+          ref.read(tabsProvider.notifier).state = tab;
+          return const MainNavigationScreen();


유저가 선택한 tab값을 동적으로 변수 tab에 할당

tabsProvider의 state에 변수 tab이 가진 값 할당

1,2로 인해 MainNavigationScreen에서 인자를 받지 않아도 되기 때문에 tab 프로퍼티 제거.

changhwan77 added 5 commits February 14, 2024 22:48

트티 홈 상단 배너 추가 및 레이아웃 수정

543ec50

feat: 메인 배너 size 조절

e5759ba

fix: 회원가입 화면 체크 박스 에러 수정

43adaae

fix: 로그아웃 시 트티 탭으로 이동시키기 위한 로직 수정

eb84761

fix: 유저가 선택한 tab이 없을 시 기본 값 설정 로직 추가

f1e51f5

changhwan77 changed the title ~~🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가~~ 🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 Feb 16, 2024

changhwan77 commented Feb 16, 2024

View reviewed changes

changhwan77 merged commit 2868c77 into master Feb 17, 2024
1 check failed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 #9

🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 #9

changhwan77 commented Feb 16, 2024 •

edited

Loading

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024

changhwan77 Feb 16, 2024


		final tab = ref.watch(tabsProvider);

		int selectedIndex = tabs.indexOf(tab);

		@@ -0,0 +1,3 @@
		import 'package:flutter_riverpod/flutter_riverpod.dart';

		final tabsProvider = StateProvider<String>((ref) => 'tuti');

🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 #9

🚧 로그아웃 시 트티 홈 탭으로 이동시키는 로직 추가 및 회원 가입 전체 동의 버튼 에러 수정 #9

Conversation

changhwan77 commented Feb 16, 2024 • edited Loading

Title

What type of PR is this?

Description

Related Issue number and link

Mobile & Desktop Screenshots/Recordings

Checklist:

To Reviewers

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

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

changhwan77 commented Feb 16, 2024 •

edited

Loading