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.

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

[#3] Feature/home #23

Merged
merged 4 commits into from
Mar 6, 2022
Merged

[#3] Feature/home #23

merged 4 commits into from
Mar 6, 2022

Conversation

mori8
Copy link
Member

@mori8 mori8 commented Mar 6, 2022

메인 페이지 UI 전면 개선

  • 환영 인사와 함께 오늘의 총 이벤트 개수 보여주기
  • 오늘의 이벤트를 진행 시간과 함께 보여주기
  • 번역/검색 화면으로 이동하는 버튼

@mori8 mori8 requested a review from hee-suh March 6, 2022 11:59
Copy link
Member

@hee-suh hee-suh left a comment

Choose a reason for hiding this comment

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

완전 예뻐 최고예요!!! 수고 많았습니당 😄

@@ -57,9 +58,18 @@ export default function App() {
name="Home"
component={HomeScreen}
options={{
headerStyle: { backgroundColor: '#333D79' },
Copy link
Member

Choose a reason for hiding this comment

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

import { theme } from './core/theme'; 해서 theme.colors.primary 로 사용 부탁~!

Copy link
Member Author

Choose a reason for hiding this comment

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

string type만 가능하다고 해서 부득이하게 저렇게 사용했습니다 ^ㅅㅠ

Copy link
Member

Choose a reason for hiding this comment

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

잘 작동해요! ForgotPasswordScreen 없애는 김에 수정해서 올릴게요~!

headerRight: () => <LogoutButton/>
headerRight: () => <LogoutButton/>,

headerTitle: (props) => ( // App Logo
Copy link
Member

Choose a reason for hiding this comment

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

로고 넘 예뻐요...

Copy link
Member Author

Choose a reason for hiding this comment

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

^ㅅ^

date: string,
notices: {
total_results: string[],
notice_body: {
id: number,
title: string,
summary: {id: number, content: string, highlight: boolean}[],
summary: {id: number; content: string; highlight: boolean}[],
Copy link
Member

Choose a reason for hiding this comment

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

어엇 쉼표가 맞지 않나요?

Copy link
Member

Choose a reason for hiding this comment

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

여기에 있던 registered가 사라져버렸는데, 이전 commit에서 이미 registered 포함한 걸로 여러 파일들 수정해줘버려서 main pull 받고 type.ts 파일에 interface UserProfile 다시 추가해줘야할 것 같슴다!

Copy link
Member Author

Choose a reason for hiding this comment

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

둘 다 가능한데 쉼표로 통일할게요!

userId: 1,
username: "Suyeon",
gmail: "kaithape@gmail.com",
profileImageType: 1,
Copy link
Member

Choose a reason for hiding this comment

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

profileImageType은 1부터 시작인가요??

Copy link
Member Author

Choose a reason for hiding this comment

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

넵! 1부터 8까지입니당

childId: 1,
childName: "Soo",
events: [{
time: "10:00",
Copy link
Member

Choose a reason for hiding this comment

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

time이 없는 경우에는 어떻게 하나요?
구글 캘린더는 오전 8시를 기본 값으로 설정해버리는데 우리도 그렇게 할까??

Copy link
Member Author

Choose a reason for hiding this comment

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

그것도 굿! 아니면 1교시 시작이 9시니까 9시로 해도 괜찮을듯!

Copy link
Member

Choose a reason for hiding this comment

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

오 고것 참 좋다

const [fontsLoaded, SetFontsLoaded] = useState<boolean>(false);
const [userProfile, setUserProfile] = useState<UserProfile>({userId: 0, username: "", gmail: "", profileImageType: 0, language: "", children: []});
const [events, setEvents] = useState<{childId: number, childName: string, events: {time: string, content: string}[]}[]>();
const [totalEventsCount, setTotalEventsCount] = useState<number>(4);
Copy link
Member

Choose a reason for hiding this comment

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

얜 나중에 루프 돌려서 계산하나요?

Copy link
Member Author

Choose a reason for hiding this comment

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

네네!

{events?.map(notice =>
<TouchableHighlight style={[styles.childButton, {
backgroundColor: nowSelectedChildId === notice.childId ? theme.colors.primary : "#ffffff",
}]} onPress={() => handleNowSelectedChildId(notice.childId)}>
Copy link
Member

Choose a reason for hiding this comment

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

얘는 () => 이렇게 했을 때 잘 먹나여?? 언제 되고 언제 안되는 거지 😳

Copy link
Member Author

Choose a reason for hiding this comment

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

yes! 하지만 언니 컴에서 안돌아갈까봐 무섭

Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ쿠ㅜ 이번엔 잘 먹어...

)}
</View>
<View style={styles.todayNoticeWrapper}>
{events.filter(notice => notice.childId == nowSelectedChildId)[0].events.map(event =>
Copy link
Member

Choose a reason for hiding this comment

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

물음표들 마구 더 추가해주면 좋을 듯 불안햄

Copy link
Member Author

Choose a reason for hiding this comment

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

오케 나도 호기심이 많아질 필요가 있다

@hee-suh hee-suh merged commit 85bdc77 into main Mar 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants