-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feature: 게시글 생성 페이지 퍼블리싱 #535 #537
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트
ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어
Labeler가 제목과 설명에 있는 특별한 텍스트와 일치하는 레이블을 적용했습니다. |
Bundle SizesCompared against none
Dynamic import: None found. |
LGTM 👍 |
guesung
pushed a commit
that referenced
this pull request
Jan 2, 2024
* feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정
dev-dong-su
added a commit
that referenced
this pull request
Jan 8, 2024
* feat : text 가운데 설정 * feat : 그룹 없을 때 화면 * feat : 매칭 - 그룹 없을 때 테스트 * refactor : issue labeler 제거 * refactor : 주석 제거 * refactor : Script 단일 태그로 수정 * refactor : 사용하지 않는 import문 제거 * feat : 이미지 업로드 필요한 경우 RN에 권한 요청 메시지 전달 * refactor : react-error-boundary에서 suspensive/react로 라이브러리 교체 * chore : 페이지 업그레이드 * refactor : deprecated 파일 (RetryErrorBoundary) 제거 * Feature: 게시글 생성 페이지 퍼블리싱 #535 (#537) * feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정 * feat: 커뮤니티 게시글 dummy mocking * refactor: 일상 톡톡 -> K-POP으로 수정 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 댓글 작성 form 추가 --------- Co-authored-by: Kuesung Park <gueit214@naver.com>
guesung
pushed a commit
that referenced
this pull request
Jan 14, 2024
* feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정
guesung
added a commit
that referenced
this pull request
Jan 14, 2024
* feat : text 가운데 설정 * feat : 그룹 없을 때 화면 * feat : 매칭 - 그룹 없을 때 테스트 * refactor : issue labeler 제거 * refactor : 주석 제거 * refactor : Script 단일 태그로 수정 * refactor : 사용하지 않는 import문 제거 * feat : 이미지 업로드 필요한 경우 RN에 권한 요청 메시지 전달 * refactor : react-error-boundary에서 suspensive/react로 라이브러리 교체 * chore : 페이지 업그레이드 * refactor : deprecated 파일 (RetryErrorBoundary) 제거 * Feature: 게시글 생성 페이지 퍼블리싱 #535 (#537) * feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정 * feat: 커뮤니티 게시글 dummy mocking * refactor: 일상 톡톡 -> K-POP으로 수정 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 댓글 작성 form 추가 --------- Co-authored-by: Kuesung Park <gueit214@naver.com>
guesung
pushed a commit
that referenced
this pull request
Jul 12, 2024
* feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정
guesung
added a commit
that referenced
this pull request
Jul 12, 2024
* feat : text 가운데 설정 * feat : 그룹 없을 때 화면 * feat : 매칭 - 그룹 없을 때 테스트 * refactor : issue labeler 제거 * refactor : 주석 제거 * refactor : Script 단일 태그로 수정 * refactor : 사용하지 않는 import문 제거 * feat : 이미지 업로드 필요한 경우 RN에 권한 요청 메시지 전달 * refactor : react-error-boundary에서 suspensive/react로 라이브러리 교체 * chore : 페이지 업그레이드 * refactor : deprecated 파일 (RetryErrorBoundary) 제거 * Feature: 게시글 생성 페이지 퍼블리싱 #535 (#537) * feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정 * feat: 커뮤니티 게시글 dummy mocking * refactor: 일상 톡톡 -> K-POP으로 수정 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 댓글 작성 form 추가 --------- Co-authored-by: Kuesung Park <gueit214@naver.com>
guesung
pushed a commit
that referenced
this pull request
Jul 17, 2024
* feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정
guesung
added a commit
that referenced
this pull request
Jul 17, 2024
* feat : text 가운데 설정 * feat : 그룹 없을 때 화면 * feat : 매칭 - 그룹 없을 때 테스트 * refactor : issue labeler 제거 * refactor : 주석 제거 * refactor : Script 단일 태그로 수정 * refactor : 사용하지 않는 import문 제거 * feat : 이미지 업로드 필요한 경우 RN에 권한 요청 메시지 전달 * refactor : react-error-boundary에서 suspensive/react로 라이브러리 교체 * chore : 페이지 업그레이드 * refactor : deprecated 파일 (RetryErrorBoundary) 제거 * Feature: 게시글 생성 페이지 퍼블리싱 #535 (#537) * feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정 * feat: 커뮤니티 게시글 dummy mocking * refactor: 일상 톡톡 -> K-POP으로 수정 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 댓글 작성 form 추가 --------- Co-authored-by: Kuesung Park <gueit214@naver.com>
guesung
pushed a commit
that referenced
this pull request
Jul 17, 2024
* feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정
guesung
added a commit
that referenced
this pull request
Jul 17, 2024
* feat : text 가운데 설정 * feat : 그룹 없을 때 화면 * feat : 매칭 - 그룹 없을 때 테스트 * refactor : issue labeler 제거 * refactor : 주석 제거 * refactor : Script 단일 태그로 수정 * refactor : 사용하지 않는 import문 제거 * feat : 이미지 업로드 필요한 경우 RN에 권한 요청 메시지 전달 * refactor : react-error-boundary에서 suspensive/react로 라이브러리 교체 * chore : 페이지 업그레이드 * refactor : deprecated 파일 (RetryErrorBoundary) 제거 * Feature: 게시글 생성 페이지 퍼블리싱 #535 (#537) * feat: ListBox 제작 ListBoxController를 상위로 ListBox ListBoxOptions를 렌더링 하는 컴포넌트 * refactor:ListBoxController의 Context API 적용 ListBox의 open 상태를 변경하기 위해 props로 내려주던걸 Context API로 제어 * feat: 게시글 생성 페이지 퍼블리싱 * feat: 게시글 작성취소 확인 모달 * refactor: 경로 축약 * refactor: 사용하지 않는 import 제거 * refactor: 컨벤션에 맞게 코드 수정 * feat: ImageSection을 공통 컴포넌트로 분리 * refactor: interface 이름 수정 * refactor: 타입 수정 * fix: modal i18n 수정 * feat: 커뮤니티 게시글 dummy mocking * refactor: 일상 톡톡 -> K-POP으로 수정 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 게시글 상세 컨텐츠 영역 퍼블리싱 * feat: 댓글 작성 form 추가 --------- Co-authored-by: Kuesung Park <gueit214@naver.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
💡 왜 PR을 올렸나요?
💁 무엇이 어떻게 바뀌나요?
상의할 부분을 함께 풀 리퀘스트에 남깁니다.
💬 리뷰어분들께
ImageSection 컴포넌트의 경우 그룹 게시글 작성 페이지에서도 사용하고 커뮤니티 게시글에서도 사용하니 중복되는 영역이 생겼습니다.
우선은 복붙으로 컴포넌트를 새롭게 생성했습니다. 따로 컴포넌트로 분리해서 제네릭 타입으로 확장을 보장하는 방법이 좋아 보이는데 어떻게 생각하시나요?
추가적으로 text input, listbox image 등의 요소들이 모여있는 컴포넌트를 InputSection으로 이름을 만들었는데 InputForm이 더 적절한 이름일까요?