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

Updated engine migration page with ShaderMaterial changes #702

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

Conversation

mvaligursky
Copy link
Contributor

extra info

@mvaligursky mvaligursky self-assigned this Feb 4, 2025
@mvaligursky mvaligursky requested a review from a team February 4, 2025 16:53
@@ -33,6 +35,10 @@ Numerous deprecated functions that provided backward compatibility have been rem

The BasicMaterial has been removed. To achieve equivalent functionality, you can use a StandardMaterial with the emissive color or emissive map set as a replacement.

#### Shader Material

If you've received error `Material class cannot be instantiated, use ShaderMaterial instead`, this indicates the need to modify your code use ShaderMaterial instead. Details description and an example can be found [here](https://github.com/playcanvas/engine/pull/6831).
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
If you've received error `Material class cannot be instantiated, use ShaderMaterial instead`, this indicates the need to modify your code use ShaderMaterial instead. Details description and an example can be found [here](https://github.com/playcanvas/engine/pull/6831).
If you've received the error `Material class cannot be instantiated, use ShaderMaterial instead`, this indicates the need to modify your code to use `ShaderMaterial` instead. Detailed description and an example can be found [here](https://github.com/playcanvas/engine/pull/6831).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants