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

[Fix][QA] 전역적으로 단순한 QA(0422) 처리 및 디버깅 #130

Merged
merged 6 commits into from
Apr 23, 2023

Conversation

watchiswatch
Copy link
Collaborator

Fix

  • 0422 노션에 등록된 QA 다발적으로 처리
  • 신고하기 바텀시트의 기타신고사유 튕김현상 디버깅

@watchiswatch watchiswatch self-assigned this Apr 23, 2023
// height: 1,
// },
// shadowOpacity: 0.25,
// shadowRadius: 1.84,
Copy link
Collaborator

Choose a reason for hiding this comment

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

아예 안쓰는 주석이면 없에주세요.

@@ -16,11 +16,11 @@ export interface InsightData {
writer: InsightWriter;
}

interface Link {
export interface Link {
Copy link
Collaborator

Choose a reason for hiding this comment

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

.d.ts 파일에서는 export를 사용하지 않습니다.

타입선언 전용 파일인데 컴파일러가 이를 인식하여 전역적으로 사용할 수 있도록 바꾸어주기 때문입니다.

export로 변경하며 다른 변경사항들이 있었을꺼같은데 롤백해주셨으면 합니다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

확인했습니다.

Copy link
Collaborator

@ysh4296 ysh4296 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@ysh4296 ysh4296 merged commit dca0bad into main Apr 23, 2023
@watchiswatch watchiswatch deleted the Fix/QA_0422 branch May 7, 2023 11:09
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