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

service/booQへのbooQ botのコメントの投稿時、ユーザー名を表示するようにする。 #504

Merged
merged 3 commits into from
Sep 1, 2021

Conversation

masa13535
Copy link
Contributor

@masa13535 masa13535 commented Jun 30, 2021

service/booQへのbooQ botのコメントの投稿時、ユーザー名を表示するようにする。
実装方法:
router/comments.goのPostComments関数の中のmessageの定義にUserIDを追加する。
46行目

message := fmt.Sprintf("### コメントを投稿しました\n%v\n%v", itemInfo, comment.Text, comment.UserID)

Close #389

@ryoha000
Copy link
Member

Close #{issue番号}

@masa13535 masa13535 changed the title first commit issue-389 Jun 30, 2021
@masa13535 masa13535 changed the title issue-389 service/booQへのbooQ botのコメントの投稿時、ユーザー名を表示するようにする。 Jun 30, 2021
@masa13535
Copy link
Contributor Author

実装しました。レビューお願いします。

@sapphi-red sapphi-red requested a review from ryoha000 July 1, 2021 02:58
router/comments.go Outdated Show resolved Hide resolved
@masa13535
Copy link
Contributor Author

遅くなりましたが修正しました。

Copy link
Member

@ryoha000 ryoha000 left a comment

Choose a reason for hiding this comment

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

LGTM

@ryoha000 ryoha000 merged commit 2b3a85d into master Sep 1, 2021
@ryoha000 ryoha000 deleted the issue-389 branch September 1, 2021 12:29
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