#648 closed defect (fixed)
Using Twitter and Identica accounts with the same username
Reported by: | kensanata | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | BitlBee | Version: | Unlisted == unsupported. |
Keywords: | ui-fix twitter identica username | Cc: | |
IRC client+version: | Client-independent | Operating System: | Linux |
OS version/distro: |
Description
I use kensanata on Twitter and kensanata on Identica. I managed to set up both accounts, using chat mode, and everything seems to work. The only strange thing is that my Identica stuff ends up in #twitter_kensanata and my Twitter stuff ends up in #chat_000.
I'm using the ui-fix branch.
Attachments (4)
Change History (12)
Changed at 2010-07-12T21:19:23Z by
Attachment: | kensanata.diff added |
---|
comment:1 follow-up: 2 Changed at 2010-07-13T00:23:04Z by
Thanks for the patch!
One thing that I find a bit unfortunate here is that the default name will also change. In most cases people will just be using Twitter, so #twitter_$username should be enough. It'd be nice to chop off the TLD, but with identi.ca that's not perfect either... :-(
comment:2 Changed at 2010-07-14T16:48:56Z by
Replying to wilmer:
One thing that I find a bit unfortunate here is that the default name will also change. In most cases people will just be using Twitter, so #twitter_$username should be enough. It'd be nice to chop off the TLD, but with identi.ca that's not perfect either... :-(
How about using the hostname, but strip off .com if that's what it ends in.
comment:3 Changed at 2010-07-14T21:49:27Z by
That may work, yeah. :-)
BTW, what about simply adding a room_name setting? Sorry for coming up with this at this point, but I've had complaints from people that the current name's too long. Having a setting like this would solve both problems.
Also, since Identi.ca is a fairly popular Twitter-compatible service, we can make the Twitter module register an "identica" service, besides Twitter itself, and give it different defaults for base_url, oauth and the channel name setting.
BTW, changing the username shouldn't be necessary since you can just use the rename command to set the nick to something sane.
comment:4 Changed at 2010-07-15T01:25:16Z by
Hm, not sure about the room_name setting. Yet another setting to change before it all works? I think my current patch does what is necessary. Having a room name setting in addition to that would be nice to have, but is no longer strictly necessary.
Having the Twitter module register an identica service with appropriate defaults would be awesome, but is a bit beyond my original goal. :)
Changed at 2010-07-15T01:26:23Z by
Attachment: | kensanata.2.diff added |
---|
twitter.com uses twitter, identi.ca stays identi.ca
Changed at 2010-07-15T01:42:00Z by
Attachment: | kensanata.3.diff added |
---|
Now with a separate identica protocol; docs are still missing
comment:5 Changed at 2010-07-15T01:44:24Z by
Ok, so adding a separate identica protocol was easier than I thought. :)
If you like it, I guess I'll write the docs as well. Just let me know...
comment:6 Changed at 2010-07-15T22:01:06Z by
Yeah, I told ya it's easy! :-)
Looks good, just a little bit code duplication that could be avoided. First, you can use something like memcpy() to copy the Twitter prpl struct, change a few fields and register.
Also, you can have one twitter_init() that checks prpl->name to figure out the right defaults for oauth and base_url? Unless you have a strong preference for your current approach..
comment:7 Changed at 2010-07-15T23:49:34Z by
I have no strong preferences. I'll take a look at memcpy and the suggested rewrite. I'm away over the week-end so perhaps this'll get delayed. Feel free to just take it and make any changes you want. :)
comment:8 Changed at 2010-07-17T15:09:27Z by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Alright, I made a few changes and submitted changeset:wilmer-ui-fix,720. Thanks!
Changed at 2010-07-20T18:17:39Z by
Attachment: | kensanata.4.diff added |
---|
suggested changes to the documentation
A proposed patch to the bug. Author: Alex Schroeder <alex@…>