Skip to content

Added trade state reload command#64

Merged
UltraProdigy merged 4 commits intomasterfrom
reload-tradestate
Feb 11, 2026
Merged

Added trade state reload command#64
UltraProdigy merged 4 commits intomasterfrom
reload-tradestate

Conversation

@cubefury
Copy link
Collaborator

@cubefury cubefury commented Feb 9, 2026

This PR adds the command /vending reload tradestate [player], allowing admins to reload a player's trade state while server is running.

  • Player argument is optional, will reload tradestate for self if run by a character, and fail with a relevant error message when run on server console.
  • Admins who want to reset the tradestate for a player can simply delete the player's tradestate file and run the command.
  • Player does not need to be online, their UUID just needs to have been cached from logging in at least once.
  • Player does not need to log off while tradestate is refreshed, although they should close out of the vending machine UI before the command is run, since exiting from the UI writes the current trade state to file and may override the desired reload (I'll add a check for this in a future patch to prevent this weird quirk)

Tested in dev env in SP and MP.

@cubefury cubefury requested a review from a team February 9, 2026 16:49
@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Feb 9, 2026
Copy link
Member

@UltraProdigy UltraProdigy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one typo really otherwise looks & functions great

Copy link
Member

@UltraProdigy UltraProdigy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💰

@UltraProdigy UltraProdigy merged commit 5ce5f53 into master Feb 11, 2026
1 check passed
@UltraProdigy UltraProdigy deleted the reload-tradestate branch February 11, 2026 16:14
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Feb 11, 2026
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.

3 participants