Skip to content

[pull] master from mangostwo:master#183

Merged
pull[bot] merged 11 commits intoWorld0fWarcraft:masterfrom
mangostwo:master
Dec 27, 2025
Merged

[pull] master from mangostwo:master#183
pull[bot] merged 11 commits intoWorld0fWarcraft:masterfrom
mangostwo:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Dec 27, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

bozimmerman and others added 11 commits December 26, 2025 19:03
The Problem:
The Auction House Bot is intended to mimic real player auction
behavior, which includes selling items commonly farmed and sold
by players.  However, the AH Bot was designed to not sell any
item that an NPC Vendor sells in any quantity. While this makes
sense, it prevents items that any Vendor sells in even very limited
quantities (such as all sorts of Leather) from every being listed.

The Fix:
Better mimic desired player auction behavior by allowing the AH Bot
to sell limited-quantity vendor items by narrowing the general
vendor item restriction to ONLY those items that vendors sell
in unlimited quantities.

The Details:
The AH Bot queries the npc_vendor table to compile a list of items
it will Not list on the AH.  These query results are narrowed by adding
'WHERE maxcount = 0', which will only capture items that vendors
sell in Unlimited quantities, thus allowing Leather to appear on the AH.
* Fixed non PCH build
  - Updated CMakeLists.txt
  - Updated GitHub actions
  - Updated non-portable / not safe localtime* methods
  - Minor refactoring on some time-to-string methods

* Fixed error from GitHub actions

* fix trailing space

---------

Co-authored-by: Antz <[email protected]>
@pull pull Bot locked and limited conversation to collaborators Dec 27, 2025
@pull pull Bot added the ⤵️ pull label Dec 27, 2025
@pull pull Bot merged commit f4d5ed5 into World0fWarcraft:master Dec 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants