Trade SC debts are placed into an appropriately named (Trade R{round}…#4373
Trade SC debts are placed into an appropriately named (Trade R{round}…#4373Mjb141 wants to merge 1 commit intoAsyncTI4:masterfrom
Conversation
…) pool by default
|
Hmmmm, I would suggest that debt that can be sent with buttons must be able to be cleared with buttons. The only way that this debt can be cleared is with a slash command. |
There was a problem hiding this comment.
Pull request overview
This PR improves the quality of life for debt tracking by introducing round-specific debt pools for Trade Strategy Card debts. When a player sends debt following the Trade SC, the debt now goes into a pool named "Trade R{round}" instead of the generic "Debt Account" pool.
Changes:
- Modified debt pool naming for Trade SC debts to include the current round number
- Updated the user-facing message to reflect the new pool name
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Both good points @bradleysigma, I'll look at those later today |
…) pool by default
Simple QoL change. "Sent 1 debt" from following trade sc goes into a pool named
Trade R<round>. Separates trade sc debt from regular trade debt, and for what few games don't always pay back trade at agenda (hidden agenda games in particular) it's nice to know where the debt came from.