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

release: 24-10-24 (v39) #2582

Merged
merged 16 commits into from
Oct 24, 2024
Merged

release: 24-10-24 (v39) #2582

merged 16 commits into from
Oct 24, 2024

Conversation

aixaCode
Copy link
Collaborator

What does this PR change?

Release v.039

How to test the changes?

  1. Launch the explorer
  2. ...

Our Code Review Standards

https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

fcolarich and others added 16 commits October 22, 2024 11:23
Signed-off-by: Ashley Canning <ashley.canning@decentraland.org>
Co-authored-by: Ashley Canning <ashley.canning@decentraland.org>
Co-authored-by: Aga <aga@besz.me>
## What does this PR change?
additional check for zero vector

doesn't produces spikes anymore
![image](https://github.com/user-attachments/assets/c75188ca-4eec-492c-943c-c12fc25eed45)

## How to test the changes?
just fast smoke test
When the cached TextMeshPro instances were reused and the new outline width equaled zero, it tried to disable the outline shader keyword which does not exist in the shader (so the previous values related to outline stayed the same and were still used). Now it also resets the values of the related properties.
* Fix for candidate state check

* Added additional check
Copy link
Contributor

@anicalbano anicalbano left a comment

Choose a reason for hiding this comment

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

🟢 Sanity check and smoke performed on Windows and working as expected. Besides our Smoke Test guide https://www.notion.so/decentraland/Smoke-Test-Alpha-1055f41146a58020a624e86a564d60a1 there was also checks for:

✔️ Create new account
✔️Pass through the onboarding
✔️Receive the emotes and wearables
✔️Get the notifications
✔️Get the Decentraland Citizen Badge
✔️Avatar personalization
✔️Minigames testing

Copy link

@Ludmilafantaniella Ludmilafantaniella left a comment

Choose a reason for hiding this comment

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

🟢 Sanity check and smoke performed on Mac and working as expected. https://www.notion.so/decentraland/Smoke-Test-Alpha-1055f41146a58020a624e86a564d60a1 there was also checks for:

  • Daily Quest
  • Minigames (Tower of Hanoi, Tile Tally, Word Search)

@aixaCode aixaCode added this pull request to the merge queue Oct 24, 2024
Merged via the queue into main with commit 81ee33e Oct 24, 2024
55 of 63 checks passed
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.

10 participants