first-timers-only-bot icon indicating copy to clipboard operation
first-timers-only-bot copied to clipboard

Specify label in tweet

Open hiimbex opened this issue 8 years ago • 8 comments

Hiya! I love the additional support for labels you've added in https://github.com/musically-ut/first-timers-only-bot/pull/3: https://github.com/musically-ut/first-timers-only-bot/blob/7438415d15e4a6f54666230a7aff46c35df44942/first_timers/first_timers.py#L10-L13

But I think it might be nice to clarify what label was assigned in the tweet. For example, an up-for-grabs issue probably isn't as new person friendly as first-timers-only.

So I'd suggest beginning the tweet with [label-name]. It would probably be a pretty simple change. Let me know what you think!

hiimbex avatar Aug 25 '17 04:08 hiimbex

This sounds like a reasonable thing. The only problem is the 140 character limit. I have had to do some gymnastics with the tweet text:

https://github.com/musically-ut/first-timers-only-bot/blob/a25545b2c14974b3c37ab504a65d9df63ab5bf7a/first_timers/first_timers.py#L66-L70

I'll keep the label information at this point and reduce the text further.

What would be better, though: [label-name] (2 extra characters) or #label-name hashtag (1 extra character)? How does Twitter handle dashes in hash-tags?

musically-ut avatar Aug 25 '17 10:08 musically-ut

I think the hashtag is much better actually, it makes use of what's built into twitter better. It'll be easier to search for on twitter too. I'm not sure about hyphens 🤔

hiimbex avatar Aug 25 '17 21:08 hiimbex

twitter don't allow the hashtags with hypens. screenshot 61 screenshot 62 While there are many forms the one with CamelCase is generally suggested for readability. I guess we can use #FirstTimersOnly as the hashtag. There is already lot of content with that tag We need a way to convert the content from hypen to CamelCase.

cg-cnu avatar Aug 26 '17 15:08 cg-cnu

CamelCase also saves some characters. :)

musically-ut avatar Aug 26 '17 15:08 musically-ut

#DoubleKill 😄

cg-cnu avatar Aug 26 '17 15:08 cg-cnu

Awesome! I'm pretty busy in the near future, so I might not get around to it, if you wanna throw on an up for grabs label?

If no one picks it up, I'd do it eventually, but I'd rather give someone more interested in learning twitter bots an opportunity to work on this.

hiimbex avatar Aug 27 '17 19:08 hiimbex

@hiimbex Old issue, but the bot is still active so wondering if I can help. I'm in both the #Beginners and #FirstTimersOnly categories... let me know if you're open to it.

My understanding of the spec is essentially: add a CamelCase hashtag at the end of each tweet specifying the relevant label (first-timers-only, low-hanging-fruit, up-for-grabs, beginners) while staying under 140 characters.

Edit: Tagging @musically-ut as well

archime avatar Sep 20 '18 19:09 archime

@archime Thanks for pitching in!

Things have evolved a bit since the issue was originally posted. As a response to the discussion in #3 , we ended up removing the support for other labels because the bot started tweeting too much:

https://github.com/musically-ut/first-timers-only-bot/blob/master/first_timers/first_timers.py#L11

Hence, the issue became a bit moot. However, since I am open to adding more labels, it is still a good idea to work on this, IMO. Hence, 👍 from my side.


Sidenotes: I think that Twitter has become a little more generous about the 140 character limit.

musically-ut avatar Sep 21 '18 15:09 musically-ut