-
Notifications
You must be signed in to change notification settings - Fork 24
Logging back in and resuming gameplay #33
Conversation
* 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
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; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated typo fix
| m_NavMeshLayer: 0 | ||
| m_StaticEditorFlags: 0 | ||
| m_IsActive: 1 | ||
| m_IsActive: 0 |
There was a problem hiding this comment.
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).
rekhoff
left a comment
There was a problem hiding this 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.
|
I'm not sure why pushing the base branch automatically closed this, sorry for the noise. |
Description
Testing