ActualHorse/CarmenSandiego
1
0
Fork 0
Code Issues 9 Pull Requests Actions Packages Projects Releases Wiki Activity

Podium scoreboards are no longer synced between clients #15

New Issue
Closed
opened 2025-07-16 02:33:02 -04:00 by ActualHorse · 1 comment
ActualHorse commented 2025-07-16 02:33:02 -04:00
Owner
Copy Link
Copy Source

Not sure what happened there, but we need to investigate this.

Not sure what happened there, but we need to investigate this.
ActualHorse added this to the v0.5 milestone 2025-07-16 02:33:02 -04:00
ActualHorse added the
Type
Bug
Priority
A
 labels 2025-07-16 02:33:02 -04:00
ActualHorse self-assigned this 2025-07-16 02:33:02 -04:00
ActualHorse commented 2025-07-16 04:22:54 -04:00
Author
Owner
Copy Link
Copy Source

It was another ownership issue. Scoring was changed to use field change callbacks for syncing to clients, but functions to change the score were being called by the game manager, which never takes ownership of an in-use podium. This is fixed now by using events called on the owner to adjust scores. This probably doesn't fix the admin panel's adjustment functions, but that can be fixed later.

It was another ownership issue. Scoring was changed to use field change callbacks for syncing to clients, but functions to change the score were being called by the game manager, which never takes ownership of an in-use podium. This is fixed now by using events called on the owner to adjust scores. This probably doesn't fix the admin panel's adjustment functions, but that can be fixed later.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Priority
A
The priority is critical
Priority
B
The priority is high
Priority
C
The priority is medium
Priority
D
The priority is low
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
Type
Breaking
Breaking change that won't be backward compatible
Type
Bug
Something is not working
Type
Documentation
Documentation changes
Type
Enhancement
Improve existing functionality
Type
Feature
New functionality
Type
Security
This is security issue
Type
Testing
Issue or pull request related to testing
No labels
Priority
A
Type
Bug
Milestone
No items
No Milestone v0.5
Projects
Clear projects
No projects
Assignees
Clear assignees
ActualHorse
No Assignees ActualHorse
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ActualHorse/CarmenSandiego#15