OpenWF/SpaceNinjaServer
28
29
Fork 34
Code Issues 25 Pull Requests 2 Actions Packages Projects Releases Activity

WIP: Fix currency synchronization and add real-time validation #2741

Closed
Infity95 wants to merge 1 commits from Infity95:fix-currency-sync-rebased into main
pull from: Infity95:fix-currency-sync-rebased
merge into: OpenWF:main
Conversation 2 Commits 1 Files Changed 3 +39 -10
Infity95 commented 2025-09-02 11:50:50 -07:00
First-time contributor
Copy Link

Problem

Rapid purchases cause currency synchronization delays, resulting in negative balances.

Solution

  • Added balance validation in updateCurrency function
  • Implemented error handling in purchase controller
  • Enabled real-time currency validation before purchases

Changes

  • src/services/inventoryService.ts: Added validateBalance parameter
  • src/controllers/api/purchaseController.ts: Added try-catch error handling
  • src/services/purchaseService.ts: Enabled validation in purchase flow

Result

Prevents negative currency balances during rapid purchase operations.

## Problem Rapid purchases cause currency synchronization delays, resulting in negative balances. ## Solution - Added balance validation in `updateCurrency` function - Implemented error handling in purchase controller - Enabled real-time currency validation before purchases ## Changes - `src/services/inventoryService.ts`: Added `validateBalance` parameter - `src/controllers/api/purchaseController.ts`: Added try-catch error handling - `src/services/purchaseService.ts`: Enabled validation in purchase flow ## Result Prevents negative currency balances during rapid purchase operations.
Infity95 added 1 commit 2025-09-02 11:50:51 -07:00
Fix currency synchronization and add real-time validation
Some checks failed
Build / build (pull_request) Failing after 1m13s
Details
75ecabcdaf
Infity95 changed title from Fix currency synchronization and add real-time validation to WIP: Fix currency synchronization and add real-time validation 2025-09-02 12:00:24 -07:00
AMelonInsideLemon commented 2025-09-02 18:47:27 -07:00
Member
Copy Link

Nice AI slop. #2740 is joke for you?

Nice AI slop. #2740 is joke for you?
Sainan commented 2025-09-02 20:22:19 -07:00
Owner
Copy Link

My AI agents at least manage to run prettier before committing.

My AI agents at least manage to run prettier before committing.
Sainan closed this pull request 2025-09-02 20:22:19 -07:00
Sainan deleted branch fix-currency-sync-rebased 2025-09-02 20:22:27 -07:00
Some checks failed
Build / build (pull_request) Failing after 1m13s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
An existing implemention is incorrect
dependencies
Pull requests that update a dependency file
documentation
Improvements or additions to documentation
enhancement
New feature or request specific to the emulator
help wanted
Extra attention is needed
improvement
missing data
part of more major rehaul
Can't be worked on right now without merge conflicts later
pr'd for
Soon™
question
Further information is requested
unimplemented
you can do this
Considered easy enough for someone's first PR.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: OpenWF/SpaceNinjaServer#2741
No description provided.
Delete Branch "Infity95:fix-currency-sync-rebased"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?