search

raisedragon / pircbotx

Automatically exported from code.google.com/p/pircbotx
0 stars 0 forks source link

Enhance Usability #219

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Rename what is currently PircBotX to Network, and rename MultiBotManager to 
PircBotX. Thus, reducing (possibly removing) confusion. Network should retain 
its connect commands so they can be connected and disconnected individually.

Original issue reported on code.google.com by jzhou2...@gmail.com on 11 Dec 2014 at 10:19

GoogleCodeExporter commented 9 years ago 
(I'm responding here cause, you're not on IRC atm)

18:32 <TheLQ> [01:36:54] JZTech101 btw about #219, I think it might be best to 
wait until 2.4 when the core gets modularized. Might as well do all the API 
breakage at once

sure. *groans at more API changes*

18:32 <TheLQ> [01:38:08] Do you think it should be prefixed with Irc though? Eg 
PircBotX->IrcNetwork, MultiBotManager->IrcManager

No. Unless you plan having PircBotX handle something other than IRC (Say wha..) 
There is no point to prefixing IRC to it

Original comment by jzhou2...@gmail.com on 15 Dec 2014 at 3:37

GoogleCodeExporter commented 9 years ago 
Ugh. I hate not being able to edit my comments >_>. Anyways, the api was 
already kinda broken when addBot was changed to addNetwork and all of the 
<pircbotx> was removed. A bit more renaming can't hurt?

Original comment by jzhou2...@gmail.com on 15 Dec 2014 at 3:41

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
I removed @deprecated from addBot but kept addNetwork as an alias. I'll decide 
weather to go through with this in 2.2

Original comment by Lord.Qua...@gmail.com on 17 Dec 2014 at 7:34