Styling/#124 #132

seonghunYang · 2024-07-10T09:47:10Z

📌 작업 내용

배포 링크: https://myongji-graduate-next.vercel.app/

구현 내용 및 작업 했던 내역

배포 파이프라인 완료
모바일, 태블릿, PC에 맞게 스타일링 수정
GNB 작업
약관 및 회원가입 안내 멘트 수정
유저 API 연동 및 테스트

🔊 도움이 필요한 부분

약관 내용이 최신화된 것이 있으면 변경해야 할 것 같아요. 회의 때 이야기해 봅시다.

추가 안내 사항

환경 변수와 이를 기반으로 한 URL 생성 01d5efa
- mock 여부에 따른 URL 변경 처리를 위해 환경변수 관리가 필요해졌습니다.
- mock 사용 시 .env.local 파일에 아래 속성을 추가하여 사용 가능하며, 서버 실행 시 mock 여부와 클라이언트 mock 여부의 상태가 다르면 에러가 발생하도록 만들었습니다.
  NEXT_PUBLIC_API_MOCKING=enable
- 추가로, 파싱 서버는 환경변수 처리가 필요 없을 것 같아서 리터럴 값으로 설정했습니다.
Funnel 오류 수정 d574585
- Funnel 사용 시 뒤로 가기가 되지 않는 문제를 해결했습니다.
refreshToken 기반 재시도 로직 작성 e0dfdbf
- 유경이의 강요로 추가했습니다.
- 미들웨어 동작이 까다로워서 작업이 생각보다 어려웠습니다.

…arger screens

…larger screens

…responsiveness

…siveness

yougyung

수고많으셨습니다. 혹시 accessToken이 만료되는 경우 refresh token으로 갱신하는 코드는 추후 개발될 예정인가요?

yougyung · 2024-07-12T07:08:03Z

app/business/api-path.ts

@@ -1,5 +1,4 @@
-const BASE_URL = 'http://localhost:9090';
-process.env.API_MOCKING === 'enable' ? 'http://localhost:9090' : '';
+const BASE_URL = 'http://staging-plzgraduation-env.eba-s4bbj5cp.ap-northeast-2.elasticbeanstalk.com/api/v1';


.env파일로 별도로 관리하지 않으신이유가 궁금해요!

yougyung · 2024-07-12T07:17:02Z

app/(sub-page)/components/navigation-items.tsx

+        <NavigationItem href={'/sign-in'} label="로그인" />
+      )}
+      <NavigationItem href={'/tutorial'} label="튜토리얼" />
+      <NavigationItem href={'/'} label="팀소개" />


팀 소개의 링크는 https://soft-anorak-0ca.notion.site/e35e3b210995463fa748f35aab536f2c?pvs=74 입니다!

yougyung · 2024-07-12T07:18:36Z

app/(sub-page)/components/navigation-items.tsx

+  );
+}
+
+type NavigationItemProps = {


convention상 interface로의 변경이 통일성 있을 것 같아요!

app/business/services/user/user.validation.ts

gahyuun

고생하셨습니다!

sign up form에서 funnel로 관리하면서 발생하게 되는 몇몇 문제점이 있네요..!
이에 대해서 회의때 이야기를 나눠보면 좋을 것 같아요:)

/sign-in에서 회원가입 클릭 후 /sign-up?funnel-step=terms 진입했을 때 뒤로 가기가 안됩니다
동의하기를 클릭하지 않고 query string 조작해서 sign-up?funnel-step=form 페이지로 이동할 수 있습니다

gahyuun · 2024-07-13T07:40:50Z

app/(sub-page)/sign-in/page.tsx

 export default function Page() {
  return (


몇몇 페이지 컴포넌트는 MyPage, ResultPage로 네이밍이 되어있고, 몇몇 페이지 컴포넌트는 별다른 이름 지정 없이 Page로 되어있네요! 통일하면 좋을 것 같아요

이 부분은 병합한 후에 통일하는 과정을 진행하도록 하겠습니다.

gahyuun · 2024-07-13T07:53:47Z

app/business/services/user/user.query.ts

+    if (isValidation(result, UserInfoResponseSchema)) {
+      return result;
+    } else if (isValidation(result, InitUserInfoResponseSchema)) {


if( isValidation(result, UserInfoResponseSchema) || isValidation(result, InitUserInfoResponseSchema) ) return result;
이렇게 처리하지 않으신 이유가 있을까요?!

리팩터링을 점진적으로 진행하면서 일단 분리한 후 다시 합치는 과정을 거치는데, 이번에는 합치게 누락된 것 같습니다!

gahyuun · 2024-07-13T07:55:03Z

app/ui/user/sign-up-form/sign-up-form.skeleton.tsx

+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />
+        <Skeleton className="h-4 w-1/2" />
+        <Skeleton className="h-4 w-1/4" />


<Skeleton className="h-4 w-1/2" /> <Skeleton className="h-4 w-1/4" />

이게 반복인 것 같은데 map을 돌리지 않고 하드코딩하신 이유가 있을까요!?

의미론적인 단위가 아니라 반복적이지만 중복이라고 생각하지 않아서 map을 사용하지는 않았습니다.

Co-authored-by: yougyung <[email protected]>

…serInfoResponse schemas

…ot provided

seonghunYang · 2024-07-16T08:13:17Z

고생하셨습니다!

sign up form에서 funnel로 관리하면서 발생하게 되는 몇몇 문제점이 있네요..! 이에 대해서 회의때 이야기를 나눠보면 좋을 것 같아요:)

/sign-in에서 회원가입 클릭 후 /sign-up?funnel-step=terms 진입했을 때 뒤로 가기가 안됩니다

동의하기를 클릭하지 않고 query string 조작해서 sign-up?funnel-step=form 페이지로 이동할 수 있습니다

퍼널 주요 문제는 아래 커밋에서 해결했습니다
d574585

seonghunYang · 2024-07-16T08:13:37Z

수고많으셨습니다. 혹시 accessToken이 만료되는 경우 refresh token으로 갱신하는 코드는 추후 개발될 예정인가요?

재시도 로직 추가 구현했습니다
e0dfdbf

gahyuun

고생하셨습니다~

gahyuun · 2024-07-16T13:21:52Z

app/utils/api/setup-url.util.ts

+  const isServerMoockingEnabled = process.env.API_MOCKING === 'enable';
+  const isClientMoockingEnabled = process.env.NEXT_PUBLIC_API_MOCKING === 'enable';
+
+  if (isMockServer()) {


수정을 요구하는 건 아니지만, isMockServer 가 선언전에 사용되어서, 순간적으로 isMockServer 함수는 어디에 있는거지? 하면서 찾았던 것 같아요! 나중에 이런 경우 화살표 함수로 작성하면 더 좋을 것 같아요 🫡

요건 사실 의도한 거긴 해요. 기본적으로 코드를 읽을 때는 최소한의 정보를 알게끔 하는 게 가독성이 좋다고 생각하는데, isMockServer 함수 선언이 앞에 있으면 구현부를 어쩔 수 없이 보게 되니까 할 수 있다면 뒤로 빼는 것을 선호합니다.

코드 리뷰를 할 때는 물론 어디에 선언되어 있는지 찾기 어려울 수 있지만, 일반적인 상황에서 코드를 읽을 때는 isMockServer 함수가 의도를 제대로 드러내면 구현부는 노출하지 않아 읽는 사람이 모르도록 하는 게 더 좋다고 생각해요.

yougyung · 2024-07-17T02:01:27Z

수고많으셨습니다 !! 한 가지 궁금한 점은 저는 서버주소를 민감한 내용으로 간주해서 작업을 진행할 때, env file에 포함시키는 편입니다. 서버 주소를 노출시켜도 괜찮다고 생각하신 이유를 여쭤볼 수 있을까요?

app/utils/api/setup-url.util.ts

Co-authored-by: yougyung <[email protected]>

seonghunYang added 30 commits June 25, 2024 11:46

chore: Add navigation buttons to navigation bar for larger screens

5a7bec2

refactor: Update navigation bar to use NavigationItem component for l…

6f47927

…arger screens

refactor: Update navigation bar to use NavigationItems component for …

173f84e

…larger screens

refactor: Update navigation bar to use NavigationItems component for …

b98b09a

…larger screens

refactor: Update navigation bar to use NavigationItems component for …

afe50f7

…larger screens

[web] refactor: Update sign-in page layout to use flexbox for better …

47fad13

…responsiveness

[web] refactor: Update sign-in page layout to use flexbox for better …

c48cb97

…responsiveness

[web] refactor: Update sign-up terms page layout and content

9ce5fbd

refactor: Update sign-up page to use SignUp component

840bbd8

refactor: Update sign-in page layout to use flexbox for better respon…

4650e12

…siveness

refactor: Update sign-up page layout and content

b850880

refactor: Update sign-up page layout to center the image

8bc64f5

refactor: Update sign-up success page layout and content

cb8b5ca

refactor: Update API base URL to staging environment

1c19971

refactor: Handle non-200 response in createUser function

625f704

refactor: Update user login error message status code

c6f2785

refactor: Update API base URL to staging environment

672bb58

refactor: Update sign-up page layout and content

60a1f13

refactor: Update sign-up page layout and content

eeefa78

refactor: Add Separator component for UI layout consistency

1ffc88f

refactor: Add buttons for account management in sign-in page

d326c5f

refactor: Update sign-in page layout and content

39269a1

refactor: Add links to sign-in page buttons for better user navigation

c35a52c

refactor: Update sign-up page layout and content

e24a9a7

refactor: Update sign-up success page layout and content

76f763f

refactor: Update user authentication API endpoint

5982e45

refactor: Update API base URL to staging environment

898dc56

refactor: Update user authentication API endpoint

78f5ddc

refactor: Remove unused validateToken function from user command

c363da3

refactor: Synchronize user command and validation code

b1d3687

seonghunYang added 7 commits July 10, 2024 06:19

refactor: Update user sign-out functionality and delete cookies

3a080d4

chore: Update API_PATH in result-handler.mock.ts

962666e

chore: Add Vercel production deployment workflow

4275317

chore: Add div wrapper for submit button in UploadTakenLecture component

70d7eef

refactor: Update heading size and padding in Manual component

7b2c0bb

chore: Update language options in sign-up form

afe89e7

refactor: Remove unnecessary comments and improve user-related functions

911c04e

seonghunYang requested review from yougyung and gahyuun as code owners July 10, 2024 09:47

yougyung reviewed Jul 12, 2024

View reviewed changes

gahyuun reviewed Jul 13, 2024

View reviewed changes

seonghunYang and others added 7 commits July 14, 2024 13:15

Update app/business/services/user/user.validation.ts

b197e63

Co-authored-by: yougyung <[email protected]>

chore: Update navigation link for team introduction

1f43663

refactor: Update NavigationItems component to use TypeScript interface

af8f899

refactor: Update user query to handle both UserInfoResponse and InitU…

e16125a

…serInfoResponse schemas

refactor: Update useFunnel hook to handle default step when step is n…

d574585

…ot provided

feat: Update API URLs and setup URL utility function

01d5efa

refactor: Add refreshToken function for handling token refresh

e0dfdbf

gahyuun previously approved these changes Jul 16, 2024

View reviewed changes

yougyung previously approved these changes Jul 17, 2024

View reviewed changes

app/utils/api/setup-url.util.ts Outdated Show resolved Hide resolved

Merge branch 'main' into styling/#124

e0d2f17

seonghunYang dismissed stale reviews from yougyung and gahyuun via e0d2f17 July 17, 2024 03:20

Update app/utils/api/setup-url.util.ts

b04114f

Co-authored-by: yougyung <[email protected]>

seonghunYang merged commit 39056f3 into main Jul 17, 2024
1 check failed

seonghunYang deleted the styling/#124 branch July 17, 2024 03:21

seonghunYang self-assigned this Aug 12, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Styling/#124 #132

Styling/#124 #132

seonghunYang commented Jul 10, 2024 •

edited

Loading

yougyung left a comment

yougyung Jul 12, 2024

seonghunYang Jul 16, 2024

yougyung Jul 12, 2024

seonghunYang Jul 16, 2024

yougyung Jul 12, 2024

seonghunYang Jul 16, 2024

gahyuun left a comment

gahyuun Jul 13, 2024

seonghunYang Jul 16, 2024

gahyuun Jul 13, 2024

seonghunYang Jul 16, 2024

seonghunYang Jul 16, 2024 •

edited

Loading

gahyuun Jul 13, 2024

seonghunYang Jul 16, 2024

seonghunYang commented Jul 16, 2024

seonghunYang commented Jul 16, 2024

gahyuun left a comment

gahyuun Jul 16, 2024

seonghunYang Jul 17, 2024

yougyung commented Jul 17, 2024

Styling/#124 #132

Styling/#124 #132

Conversation

seonghunYang commented Jul 10, 2024 • edited Loading

📌 작업 내용

🔊 도움이 필요한 부분

추가 안내 사항

yougyung left a 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

gahyuun left a 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

seonghunYang Jul 16, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

seonghunYang commented Jul 16, 2024

seonghunYang commented Jul 16, 2024

gahyuun left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yougyung commented Jul 17, 2024

seonghunYang commented Jul 10, 2024 •

edited

Loading

seonghunYang Jul 16, 2024 •

edited

Loading