ajones05 / seearound.me-ionic

0 stars 0 forks source link

Alerts API: number doesn't match #167

Closed ajones05 closed 7 years ago

ajones05 commented 7 years ago

@yuriyua If the total number of votes is 5, then alert should say "[name] and four others liked your post." But it often has just one name, like "[name] liked your post."

yuriyua commented 7 years ago

Done

Yuriy Kurilyuk liked your post
Yuriy Kurilyuk and one other liked your post
Yuriy Kurilyuk and five others liked your post

etc

ajones05 commented 7 years ago

Still not right. For example:

image

Brings me to the original post:

image

yuriyua commented 7 years ago

fixed

yuriyua commented 7 years ago
"message": "Mory and thirty-five others liked your post",
"message": "Jess and thirteen others liked your post",
"message": "Shayane Smith and sixty-two others liked your post",
"message": "Bill and six others liked your post",
"message": "Glen and six others liked your post",
"message": "Benjamin and two others liked your post",
"message": "Aethon Blackfyre started following you"
"message": "Eileen and eleven others liked your post",
"message": "Shayane Smith and twenty-seven others liked your post",
"message": "Tisbury and ten others liked your post",
ajones05 commented 7 years ago

Seems OK in some cases, but not all. For example: image

yuriyua commented 7 years ago

FIXED

Yuriy Kurilyuk and eighteen others liked your post

2017-02-01_16h47_16