search

denielchiang / line_reminder

Reminder using LINE Notify
1 stars 0 forks source link

Multiple notification sending #88

Open denielchiang opened 1 week ago

denielchiang commented 1 week ago

Its a bug in cron job that cause multiple messages has been sent.

denielchiang commented 1 week ago

Took a quick peak today, it seems like the way it handles concurrent sending isn't correct.

The current calling is make the receiver querying multiple times and currently. It has to be a list that pass to the function doing currently calling.

denielchiang commented 4 days ago

Haven't have time to look into it for now. But it goes quite perfect in test Line chat group. Except the stick stamp won't change tho.