Like / unlike any post (or comment?)

mrienstra commented 10 years ago

Commit 7db550a771 added "unlike", as well as a crude "pending" state.

Still outstanding: like/unlike for post.jsx.

mrienstra commented 10 years ago

Still outstanding: like/unlike comments (post.jsx).

mrienstra commented 10 years ago

Commit 1a7e7632c6 added liking/unliking of comments.

@clearhythm, I think we're ready to close this, can you review functionality for completeness?

mrienstra commented 10 years ago

Current behavior:

Like/unlike any post or comment (currently this means any on PostsScreen & PostScreen).
Result of the above is immediately (optimistically) shown by toggling the heart icon state as well as the displayed likeCount. The icon & likeCount also receive pending class which causes them to pulse.
A refresh is triggered when like/unlike has been submitted successfully.
In the case of PostScreen, PostsScreen refresh is also triggered (so it will be up-to-date with changes if user uses "Back" button). Technically we are doing this PostsScreen refresh unnecessarily from PostScreen for a comment like/unlike (comment likes aren't displayed on PostsScreen).

I guess what's missing is error feedback when like/unlike was not submitted to FB successfully (e.g. if network connection is unavailable). Completion of this is blocked by #13.

mrienstra commented 10 years ago

Commit 299f3987b3 is related to this issue.

mrienstra / barchat

Like / unlike any post (or comment?) #8

Current behavior: