Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved handling of class tokens #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

httpsKingPie
Copy link

@httpsKingPie httpsKingPie commented Oct 10, 2022

Proposed method of ReplicaService.ReturnClassToken as a way of checking whether a ClassToken already exists
Proposed clearing class token names when replicas are destroyed, because class tokens are supposed to be unique to the replica and the token is not used when the replica is destroyed

(Context: use case that inspired this was a project where replicas are assigned to the player when they join ex: ClassToken = ReplicaService.NewClassToken(Replica_[playername])" and when they leave the replica is destroyed. If the player rejoins the game, an error occurs, because the class token already exists. Two proposed methods, one to check if a class token exists by retrieving it and the other to remove class tokens from internal table when replicas are destroyed)

Proposed method of ReplicaService.ReturnClassToken as a way of checking whether a ClassToken already exists
Proposed clearing class token names when replicas are destroyed, because class tokens are supposed to be unique to the replica and the token is not used when the replica is destroyed

(Context: use case that inspired this was a project where replicas are assigned to the player when they join ex:  ClassToken = ReplicaService.NewClassToken(Replica_[playername])" and when they leave the replica is destroyed. If the player rejoins the game, an error occurs, because the class token already exists.  Two proposed methods, one to check if a class token exists by retrieving it and the other to remove class tokens from internal table when replicas are destroyed)
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.

1 participant