Skip to content
This repository was archived by the owner on Aug 8, 2025. It is now read-only.

Conversation

@jdetter
Copy link
Collaborator

@jdetter jdetter commented Jul 25, 2025

Description

  • Previous to this PR if you login, create a player, then logout and log back in you are prompted for a username again instead of just resuming your gameplay. This is confusing to players because it seems like your token is not being reused and you are just creating a new player.

Testing

  1. Clear your token, then open the game, create a player. Then logout
  2. Without clearing your token, log back into the game.
  3. Make sure that:
  • Your username is correct
  • Your circles are where you left them (and the amount of circles is correct.)

jdetter and others added 30 commits November 8, 2023 09:42
* Upgraded module to 0.12

* 0.12 upgrade is working

* Fixes

* Fixed some server warnings

* Small fix

* Apply suggestions from code review

* Update Cargo.toml

* Package upgraded to 0.12.0

* Renamed SampleScene => Main

---------

Co-authored-by: John Detter <no-reply@boppygames.gg>
Co-authored-by: Tyler Cloutier <cloutiertyler@users.noreply.github.com>
* First pass of moving C# SDK Testsuite here

* Keeping scene name as Main

* Don't use local reference for C# package

* Update client/Assets/Scripts/GameManager.cs

* Several fixes, testsuite passing

* This can be merged post 1.0.0-rc1

Small fix

---------

Co-authored-by: John Detter <no-reply@boppygames.gg>
* upgrade to 1.0.0-rc1

* fixed compilation errors

---------

Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
Co-authored-by: Tyler Cloutier <cloutiertyler@aol.com>
* Fix scheduled tables

The API has changed upstream and requires user to specify those columns manually.

* Regenerate and fix paths
The attribute no longer exist, we now either infer the name automatically or specify it within the `table(scheduled(at = ...))` attribute instead.
* Circle splitting and recombining;
Client re-coloring

* Renamed Entity.id into Entity.entity_id

* Client refactor; Polish; Bugfixes

* Store username between game runs

* Renamed EntityActor.OnUpdate to OnEntityUpdate

* [WIP] C# server

* [WIP] C# server

* Changed `DateTimeOffset` to `long` in C# module

* Parallax background

* Animate entities when they get consumed

* Fixed respawn button

* Draft into Steve's branch (#12)

* Small changes

* Changes to align with the tutorial.

* Fixed rebase issue

* Merged ConnectionManager and EntityManager into GameManager for tutorial simplicity

* Renaming Actor -> Controller for simplicity's sake, although Actor is a good name

* actor -> entityController

* Merged ArenaController into GameManager for simplicity and cleaned up a few things

* Small cleanup to keep it in line with the tutorial

* Small changes to how overlapping works

* Rebrand to Blackholio

* Rebrand to Blackholio (missed one)

* Added a README.md

* fixed readme

---------

Co-authored-by: Steve Gibson <steve@clockwokrlabs.io>
Co-authored-by: Tyler Cloutier <cloutiertyler@users.noreply.github.com>
Co-authored-by: Tyler Cloutier <cloutiertyler@aol.com>
…testing (#13)

* Test with upstream subscription API + fix respawns + add suicide for testing

* Add comment on not loading Auth Token for testing.

* Remove dangling meta file?

* Switch server-csharp to local build temporarily & fix all issues

* Remove needless comment

* Add comment to csproj

* Remove debug logs
rekhoff and others added 12 commits January 24, 2025 08:03
Merged the player and logged_out_player tables into only a player table, to match the way it is used in the tutorial documentation. Slight changes to how logged_out_player was being referenced was required due to the tables having different architecture.
* Fix C# server build process & get rid of dead link in client manifest.json

* Fix unfinished paste

* Update for new c# codegen

* Update generated code and package jsons

* Revert server-rust dependency override
…19)

* Fix C# server bug with overlapping circles

Update for Timestamp stuff

* Update for EventContext changes (TODO: update package .jsons once c# SDK PR is merged

* Regenerate

* Fix package .jsons

* Update for new AuthToken logic

* Update for no onUnhandledReducerError + fix package jsons

* Revert server-rust/Cargo.toml change
These changes match those found in the Docs side with PR clockworklabs/spacetime-docs#169
* Fix version and old property

Changed version from 1.0.0-rc4 to 1.0.0
Replaced ctx.CallerIdentity to ctx.Sender

* Regenerated autogen files with actual version of spacetimedb
Changed package from stage to regular

* Create global.json

Added global.json, to prevent trying to generate code with wrong net sdk version

* Updated manifest with dependency address used in quickstart guide

---------

Co-authored-by: rekhoff <r.ekhoff@clockworklabs.io>
Co-authored-by: Zeke Foppa <bfops@users.noreply.github.com>
* Add instructions to README

* Update README.md
protected float LerpTime;
protected Vector3 LerpStartPosition;
protected Vector3 LerpTargetPositio;
protected Vector3 LerpTargetPosition;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Unrelated typo fix

@jdetter jdetter requested a review from rekhoff July 25, 2025 19:33
m_NavMeshLayer: 0
m_StaticEditorFlags: 0
m_IsActive: 1
m_IsActive: 0
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

By default now the elements underneath the username chooser are disabled by default and then are only enabled if the user has to created a new player (technically they have to choose a username).

Copy link
Contributor

@rekhoff rekhoff left a comment

Choose a reason for hiding this comment

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

Reviewed all code changes, they all look good and make logical sense to the goal. This is a great addition to improve understanding on tokens for new users using this as an example of baseline. Approved.

@bfops
Copy link
Collaborator

bfops commented Aug 5, 2025

I'm not sure why pushing the base branch automatically closed this, sorry for the noise.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants