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 mention bug #53

Merged
merged 1 commit into from
Jul 11, 2020
Merged

Fix mention bug #53

merged 1 commit into from
Jul 11, 2020

Conversation

hsp9513
Copy link
Contributor

@hsp9513 hsp9513 commented Jul 11, 2020

멘션을 사용하는 경우 멘션 대상의 닉네임 또는 일반 String에 @가 들어가게 되면 버그가 발생합니다.
만약 @가 들어가있다면 해당 갯수만큼 mentionCount를 증가시켜주어야 합니다.

증상 참고용 캡쳐이미지 입니다. https://imgur.com/a/9lkVxEW

@storycraft
Copy link
Owner

ChatMention에는 ChatUser객체가 아닌 UserInfo 객체가 들어가야합니다

@hsp9513
Copy link
Contributor Author

hsp9513 commented Jul 11, 2020

아 이것도 바뀌었군요.
하지만 UsefInfo를 사용하더라도 Unknown이 원래 닉네임으로 바뀔뿐 증상은 같습니다.

@storycraft storycraft merged commit fc82c87 into storycraft:dev Jul 11, 2020
@storycraft storycraft mentioned this pull request Jul 11, 2020
50 tasks
@hsp9513 hsp9513 deleted the mentionFix branch July 15, 2020 11:50
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.

None yet

2 participants