Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

Update GameObjectCreatorFromMetadata.cs #681

Closed
wants to merge 1 commit into from

Conversation

filod
Copy link

@filod filod commented Jan 11, 2019

it's nice to have a chance to modify prefab before Instantiate.

Since you are not accept public PR, you can take this as a feedback and close it.

it's nice to have a chance to modify prefab before Instantiate.

Since you are not accept PR, you can take this as a feedback.
@improbable-prow-robot improbable-prow-robot added the size/XS Denotes a PR that changes 0-14 lines, ignoring generated files. label Jan 11, 2019
@jamiebrynes7 jamiebrynes7 added S: being-considered Status: This issue is being considered. T: enhancement Type: This is an improvement to an existing feature A: game-object-creation Area: Gameobject creation feature module labels Jan 16, 2019
@jessicafalk
Copy link
Contributor

Thank you for your contribution!

We like the idea and have discussed this internally. Instead of passing in an extra parameter, we are considering making the methods in the GameObjectCreatorFromMetadata virtual. This would allow you to inherit from the GameObjectCreatorFromMetadata and modify it based on whatever changes you would like to make.

What do you think about this?

@jessicafalk jessicafalk added S: needs-info Status: This requires more information and removed S: being-considered Status: This issue is being considered. labels Jan 16, 2019
@filod
Copy link
Author

filod commented Jan 16, 2019

That will do too.

@jessicafalk
Copy link
Contributor

I created #687 to track progress of this feature.
Closing this PR for now.

@jessicafalk jessicafalk added S: accepted Status: This issue has been accepted. and removed S: needs-info Status: This requires more information labels Jan 16, 2019
jessicafalk pushed a commit that referenced this pull request Nov 15, 2019
* Error if blueprint RPCs have by ref arguments

* Add known issue

* Update SpatialGDK/Documentation/known-issues.md

Co-Authored-By: improbable-valentyn <32096431+improbable-valentyn@users.noreply.github.com>
jessicafalk pushed a commit that referenced this pull request Nov 15, 2019
* Error if blueprint RPCs have by ref arguments

* Add known issue

* Update SpatialGDK/Documentation/known-issues.md

Co-Authored-By: improbable-valentyn <32096431+improbable-valentyn@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A: game-object-creation Area: Gameobject creation feature module S: accepted Status: This issue has been accepted. size/XS Denotes a PR that changes 0-14 lines, ignoring generated files. T: enhancement Type: This is an improvement to an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants