You must login to post a comment. Don't have an account? Register to get one!

• 

  • Reply
  • #3

  Zidomo May 23, 2011 at 06:41 UTC - 0 likes

  Tracking trade cooldowns requires using CPU time; all mods I've tested that track them use some. Unlike ProfessonsVault at this time, which uses 0 CPU on idle.

  As there are at least two working, decent LDB trade cooldown tracking mods out there (the aformentioned Broker_TradeCooldowns as well as Ara Broker Tradeskills), I vote to not add it. Or if you ever decide to add it, have it in a seperate LOD module that can be completely disabled for those not wanting to track cooldowns (in an ACP raid mod set, for instance).

• 

  • Reply
  • #2

  oscarucb Nov 09, 2010 at 01:16 UTC - 0 likes

  Yes, I'm kind of torn about adding this feature. I personally run Broker_TradeCooldowns alongside ProfessionsVault and the two complement each other nicely, so I don't really see much point in re-creating all the functionality of Broker_TradeCooldowns within this addon. The only thing I'd probably fix is the cooldown detection of Broker_TradeCooldowns is pretty kludgy and could be made smarter, but I should probably just contribute some improvements to that project :)

  I'll leave this open for now and see how many votes it gets.

  Author/Maintainer of BigBrother, ProfessionsVault, InspectFix, RetardCheck, RoleIcons, SavedInstances, and others..

• 

  • Reply
  • #1

  Freydis88 Nov 08, 2010 at 20:37 UTC - 0 likes

  Maybe you could sort off implement the existing Ace3 addon Broker_TradeCooldowns.
  Just a thaught, because Zhinjio's addon runs perfectly already, and also is customizable in various ways.
  

  Cheers,

  Freydis88 :)
  

  George Orwell:

  
  

  "Political language is designed to make lies sound truthful and murder respectable,

  and to give the appearance of solidity to pure wind."