search

bilalghalib / Awareness

A script that validates car bomb events in Iraq
12 stars 3 forks source link

Prevent @replies from being tweeted out to be validated #5

Open bilalghalib opened 10 years ago

bilalghalib commented 10 years ago

Right now @replies are being sent out to be validated and we wind up getting our own stuff back. We could do a simple test to see if the first char is a @ and not include it. We should also not use any of @weaware tweets.

mgrube commented 10 years ago

We can add a step to add our own status to 'validated' and actually check the validated table before we tweet.

On Mon, Jun 30, 2014 at 6:20 PM, Bilal Ghalib notifications@github.com wrote:

Right now @replies are being sent out to be validated and we wind up getting our own stuff back. We could do a simple test to see if the first char is a @ and not include it. We should also not use any of @weaware tweets.

— Reply to this email directly or view it on GitHub https://github.com/bilalghalib/Awareness/issues/5.

bilalghalib commented 10 years ago

I've added a new column called isPublished which checks to see if an event was already posted. we should probably do something similar to see if an event was already sent out to this user so we don't double up?

mgrube commented 10 years ago

Sounds good to me.

On Mon, Jun 30, 2014 at 9:19 PM, Bilal Ghalib notifications@github.com wrote:

I've added a new column called isPublished which checks to see if an event was already posted. we should probably do something similar to see if an event was already sent out to this user so we don't double up?

— Reply to this email directly or view it on GitHub https://github.com/bilalghalib/Awareness/issues/5#issuecomment-47607080.

mgrube commented 10 years ago

Check my repo. Quick fix implemented in awareness_alert.