Skip to content

[WIP] Fix blockconfig limits for donations#439

Merged
tastybento merged 7 commits into
developfrom
copilot/fix-blockconfig-limits
May 16, 2026
Merged

[WIP] Fix blockconfig limits for donations#439
tastybento merged 7 commits into
developfrom
copilot/fix-blockconfig-limits

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 15, 2026

  • Fix IslandLevelCalculator.tidyUp() to apply blockconfig limits when computing donated points contribution to island level
  • Fix IslandDonateCommand.performHandDonation() to cap donation at remaining limit capacity
  • Fix IslandDonateCommand.performInvDonation() to cap per-material donation at remaining limit and return excess blocks to inventory
  • Add island.donate.limit-reached locale message to en-US.yml
  • Add tests for limit enforcement in hand and inv donation, and level calculation

Copilot AI linked an issue May 15, 2026 that may be closed by this pull request
Agent-Logs-Url: https://github.com/BentoBoxWorld/Level/sessions/e64dd47d-1dfc-4c51-967f-0646c6d8af1b

Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com>
@tastybento
Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

…nfig-limits

Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 16, 2026

@copilot resolve the merge conflicts in this pull request

Merged master into the branch (commit 821a79b). No conflicts occurred — the merge resolved cleanly.

@tastybento tastybento marked this pull request as ready for review May 16, 2026 03:56
@sonarqubecloud
Copy link
Copy Markdown

@tastybento tastybento merged commit 45f32b7 into develop May 16, 2026
3 checks passed
@tastybento tastybento deleted the copilot/fix-blockconfig-limits branch May 16, 2026 16:34
tastybento added a commit that referenced this pull request May 16, 2026
…mpts

PR #439 added limits to /is donate hand and inv execution, but the GUI
panel still accepted any amount and both the hand and inv confirm
prompts quoted pre-limit numbers. Apply limits in DonationPanel preview
and confirm (returning excess to the player), and rebuild the hand/inv
prompts to quote the actual amount that will be destroyed. Add a
multi-line limit-notice shown when any material is capped, translated
into all 16 locales.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.

blockconfig limits not working with donation.

2 participants