Resolve youpin-city/youpin-bot#8 #29
Conversation
…he first step Add buttons to report a new image or change language
tvirot
left a comment
tvirot
left a comment
There was a problem hiding this comment.
Instead of checking if(!context.isEnglish), you should use this pattern:
m.sendText(userid, context.__("First, can you send me photos or videos of the issue you found?"))
and add update the locale files.
See https://github.com/youpin-city/youpin-bot#update-localizations
youpin.js
Outdated
| context.lastSent = (new Date()).getTime(); | ||
| context.state = STATE_WAIT_INTENT; | ||
|
|
||
| greet( userid, context ); |
There was a problem hiding this comment.
You don't need this anymore, right?
youpin.js
Outdated
| if (!context.isEnglish) { | ||
| m.sendText(userid, `สวัสดีฮ่ะ คุณ ${profile.first_name} ช่วยพินสถานที่ที่พบปัญหา โดยการแชร์ ` + | ||
| 'location จาก Messenger App บนมือถือของคุณด้วยฮ่า'); | ||
| m.sendButton(userid, 'หรือมีอย่างอื่นให้ป้ายุพินช่วยจ๊ะ', |
There was a problem hiding this comment.
You might need to wrap this with another setTimeout((), 1000) to ensure that the buttons are sent after the greeting message.
BTW, I really like this solution 😄
youpin.js
Outdated
|
|
||
| } else { | ||
| context.state = STATE_WAIT_INTENT; | ||
| if (context.isEnglish) { |
There was a problem hiding this comment.
The current implementation of greet() already handle i18n for you. See https://github.com/youpin-city/youpin-bot/blob/master/youpin.js#L60
|
I hope it works now. Somehow I cannot get YouPin API up so I haven't really tested the code :( Sorry |
3 participants
Resolve #8
If the first message is a photo that is not a sticker, it will skip the first step.
It also adds buttons to report a new image or change language.