Skip to content

Conversation

@OnestarLee
Copy link
Contributor

Fixed type errors when 'strictNullChecks' is 'true'.

슬랙으로 공유드린 내용 제외하고, 수정해도 특별히 문제 없을 만한 부분 우선 수정하였습니다.

@OnestarLee OnestarLee requested review from HoonBaek, bang9, chohongm and chrisallo and removed request for HoonBaek and bang9 April 3, 2024 06:14
@OnestarLee OnestarLee self-assigned this Apr 3, 2024
@OnestarLee OnestarLee requested a review from AhyoungRyu April 3, 2024 06:15
Copy link
Collaborator

@chrisallo chrisallo left a comment

Choose a reason for hiding this comment

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

양이 엄청 많은데 고생 많으셨습니다!

Copy link
Collaborator

@chrisallo chrisallo left a comment

Choose a reason for hiding this comment

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

lgtm!

@OnestarLee OnestarLee merged commit 054f88c into feat/v3/CLNP-2496-strict-typescript Apr 5, 2024
Copy link
Contributor

@HoonBaek HoonBaek left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for this huge job!!

setMembers(members);
setHasNext(memberUserListQuery.hasNext);
setChannelUpdateId(uuidv4());
setChannelUpdateId?.(uuidv4());
Copy link
Contributor

Choose a reason for hiding this comment

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

This line should be removed (maybe conflict)

@OnestarLee OnestarLee deleted the feat/v3/strict-typescript/CLNP-2687-strict-typescript3 branch April 7, 2024 23:38
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.

4 participants