-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
giraffate
commented
Sep 27, 2018
- furikライクにメッセージ出力するためのサンプル実装を書きました。
- title, eventTypeは適当なメッセージを一旦出力しています。
- サンプル実装なのでハードコーディングしてるところもあります。
dd68e4b
to
0d0dc0b
Compare
GetEventsWithGrouping内でParsePayloadして返却型は[]interface{}にして、使う側では型アサーションすればいいのではないかと考えています と思ったけど、型アサーションすると使う側で |
main.go
Outdated
for _, event := range events { | ||
var ( | ||
eventType, title, link string | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これ、()なくていいんだっけ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これ以外はいったん良さそうです〜
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
なくていいですね、修正しますー
@shimoju #12 (comment) の件はリファクタなので後でやろうかと。Issueとして残しておきます。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🆗