-
Notifications
You must be signed in to change notification settings - Fork 614
Added the Blackholio Unreal Blueprint Tutorial #3313
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
base: master
Are you sure you want to change the base?
Conversation
The Unreal documentation has been updated but I need to fix up the Unity documentation to use signed integers as well as fix up the Blackholio demos all to use signed integers. |
…m/clockworklabs/SpacetimeDb into jlarabie/blackholio-blueprint # Conflicts: # demo/Blackholio/client-unreal/Source/client_unreal/Private/ModuleBindings/SpacetimeDBClient.g.cpp # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/SpacetimeDBClient.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/CircleRecombineTimerType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/CircleType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/ConfigType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/ConsumeEntityTimerType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/EntityType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/FoodType.g.h # demo/Blackholio/client-unreal/Source/client_unreal/Public/ModuleBindings/Types/PlayerType.g.h
Ready for review but not pushed to staging yet as we'll need to discuss the images and the new ::: setup I've used |
|
||
### Create the GameManager Actor | ||
|
||
:::server-cpp |
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.
The C++ and Blueprint documentation toggle all appear to use server-cpp
and server-blueprint
. This seems to imply the server either uses C++ or Blueprints to run, but contain instructions pertaining to the client. We likely want to change these to client-cpp
and client-blueprint
or possibly client-ucpp
to indicate this is Unreal C++ and not general C++.
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.
100% I'm actually working on this right now as I've got it running locally and :::server-rust is competing with :::server-cpp as an example. Even changing to :::client-cpp isn't working so I'm diagnosing now!
If we are creating a Blueprint project, should we still generate the project as a C++ as shown here SpacetimeDB/docs/docs/unreal/part-1.md Line 44 in 5420d14
If we should still select C++ in the Project Defaults, we may want to have a note or something, so folks going down the Blueprint path, don't generate the project as a Blueprint project out of habit. |
Yes we will want all projects to start as C++, the reason is we're generating C++ code from the module into the project so might as well start them off on the right foot. I'll make sure to get an extra note in there! |
Here are some items I found while reviewing: In SpacetimeDB/docs/docs/unreal/part-2.md Line 729 in deedc10
SpacetimeDB Build node to become available.
In SpacetimeDB/docs/docs/unreal/part-3.md Line 708 in deedc10
BP_GameMode -> BP_GameManager Should be:Open BP_GameManager and update to the following:
SpacetimeDB/docs/docs/unreal/part-3.md Line 1507 in deedc10
Diameer -> Diameter . Should read:- Add Output as Diameter with Float as the type.
SpacetimeDB/docs/docs/unreal/part-3.md Line 1560 in deedc10
Output setup task. Below screenshot should read:- Add Output as GameManager with BP Game Manager as the type.
SpacetimeDB/docs/docs/unreal/part-3.md Line 1586 in deedc10
Input -> Output . Should read:- Add Output as Center with Vector as the type.
SpacetimeDB/docs/docs/unreal/part-3.md Line 1626 in deedc10
Get Username function required Exec.![]() I put it's execution between Cast to WBP_Nameplate and Update Text in my example.Note: Editor Version: 5.6.1-44394996+++UE5+Release-5.6 SpacetimeDB/docs/docs/unreal/part-3.md Line 2055 in deedc10
Class Reference -> Object Reference . Should read:\ - Change Variable Type to BP Game Manager -> Object Reference SpacetimeDB/docs/docs/unreal/part-3.md Line 2057 in deedc10
Class Reference -> Object Reference . Should read:\ - Change Variable Type to BP Player Pawn -> Object Reference SpacetimeDB/docs/docs/unreal/part-3.md Line 2064 in deedc10
Output setup task. Below screenshot should read:- Add Output as GameManager with BP Game Manager as the type.
SpacetimeDB/docs/docs/unreal/part-3.md Line 2141 in deedc10
Bind Event to On Username Changed node.When following the server-blueprint instructions, I don't see a SetUsername being created.
SpacetimeDB/docs/docs/unreal/part-3.md Line 2149 in deedc10
|
Description of Changes
Closes: https://github.com/clockworklabs/SpacetimeDBPrivate/issues/2073
API and ABI breaking changes
None
Expected complexity level and risk
1 - Documentation update, does update the server side
Testing