Skip to content

Fix trading overflow - #1969

Open
ottml wants to merge 3 commits into
Return-To-The-Roots:masterfrom
ottml:fix_trading_overflow
Open

Fix trading overflow#1969
ottml wants to merge 3 commits into
Return-To-The-Roots:masterfrom
ottml:fix_trading_overflow

Conversation

@ottml

@ottml ottml commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Steps to reproduce:

  • Build two storehouses
  • Both storehouses have figures with one type, for example woodcutter
  • Both storehouses have 2 woodcutters
  • Send 1 woodcutter to ally storehouse
  • Then 3 woodcutters are sent to the ally instead of 1

@ottml

ottml commented Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

@Flamefire @Flow86 Could it be that some permissions are missing for pullrequests from a fork? Because the ci above is not starting because of

2 workflows awaiting approval

@ottml
ottml force-pushed the fix_trading_overflow branch from a9c088c to 2b296a3 Compare August 17, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant