buckket / twtxt

Decentralised, minimalist microblogging service for hackers.
http://twtxt.readthedocs.org/en/stable/
MIT License
1.92k stars 79 forks source link

Store tweet hashes and check them before tweeting to prevent duplicates. #40

Closed Samathy closed 8 years ago

Samathy commented 8 years ago

Maybe tweet hashes could be implemented in the meta-data of a tweet in the future. But for now, store it in another file.

jomo commented 8 years ago

I was thinking about hashing the complete line (i.e. including the time stamp) as an identifier for tweets, which then could be used to reply to a specific tweet.

Samathy commented 8 years ago

Timestamps do get hashed, along with the tweet data. I just checked!

timofurrer commented 8 years ago

Why is it a problem to tweet something multiple times? This implementation could lead into performance issues when you have a lot of tweets.

buckket commented 8 years ago

I agree with @timofurrer. Don’t see why this is problem that needs special attention right now.

Samathy commented 8 years ago

It doesnt need special attention, I guess I'm just trying to contribute some stuff! I appreciate your standing.

buckket commented 8 years ago

Your contribution is much appreciated, but right now I want to get the other things (mentions, more tests) done first, while increasing the complexity of the code base as little as possible. I’m closing this for now, and may come back to it later. Hope that’s fine with you. (ノ゚-゚)ノ