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

Append chat ngay #48

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

NamPNQ
Copy link
Contributor

@NamPNQ NamPNQ commented Aug 15, 2013

No description provided.

@andy-pham
Copy link
Contributor

Trường hợp pending > 1 cái thì sao?

@NamPNQ
Copy link
Contributor Author

NamPNQ commented Aug 15, 2013

vẫn bt mà a, mỗi cái boxchat cái 1 cái link action khác nhau, e lấy nó làm id _tmpchat nên đâu bị trùng

@andy-pham
Copy link
Contributor

Ý anh là vẫn box chat đó, nhưng người ta gõ liên tục vài messages, các messages đó bị pending thì lúc nhận response về mấy cái replace sợ bị nhầm không

Andy Pham
http://play.jupo.com/
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Thursday, August 15, 2013 at 11:39am, NamPNQ wrote:

á

@NamPNQ
Copy link
Contributor Author

NamPNQ commented Aug 15, 2013

trường hợp này đâu xảy ra đc, khi a gủi đi 1 tin, thì a disable cái form đó rồi, trừ khi người dùng cố tính thôi 💃

@andy-pham
Copy link
Contributor

Anh muốn append chat message ngay để unlock cái form đó, chứ lock như hiện tại thì hơi bất tiện

Andy Pham
http://play.jupo.com/
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Thursday, August 15, 2013 at 11:48am, NamPNQ wrote:

trường hợp này đâu xảy ra đc, khi a gủi đi 1 tin, thì a disable cái form đó rồi, trừ khi người dùng cố tính thôi


Reply to this email directly or view it on GitHub (#48 (comment)).

@NamPNQ
Copy link
Contributor Author

NamPNQ commented Aug 15, 2013

z e đã có giải pháp rồi 🎯 tạo cái _tmpid nữa, gửi lên server, server trả về kèm cái _tmpid nữa, thế đc k a

@andy-pham
Copy link
Contributor

Như trong chức năng Reminders, anh tạo id ở client side, rồi lúc nhận response thì replace tương ứng, em xem qua thử xem:
https://github.com/juposocial/jupo/blob/develop/src/templates/reminders.html#L61

Andy Pham
http://play.jupo.com/
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)

On Thursday, August 15, 2013 at 12:04pm, NamPNQ wrote:

z e đã có giải pháp rồi tạo cái _tmpid nữa, gửi lên server, server trả về kèm cái _tmpid nữa, thế đc k a


Reply to this email directly or view it on GitHub (#48 (comment)).

@NamPNQ
Copy link
Contributor Author

NamPNQ commented Aug 15, 2013

Đã xong 💃

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