Skip to content

feat: Adapt Unity side to the new Archipelago realms system#841

Merged
sandrade-dcl merged 8 commits intomasterfrom
feat/add-support-layerless-realm-unity-side
Jul 27, 2021
Merged

feat: Adapt Unity side to the new Archipelago realms system#841
sandrade-dcl merged 8 commits intomasterfrom
feat/add-support-layerless-realm-unity-side

Conversation

@sandrade-dcl
Copy link
Copy Markdown
Contributor

@sandrade-dcl sandrade-dcl commented Jul 23, 2021

What does this PR change?

Fix #804

Layers are being deprecated inside the catalysts and will be replaced by the concept of island, which should be transparent for the unity-renderer. Determine which changes need to be done in order to assimilate this changes.

How to test the changes?

Test realms with the new Archipelago approach (ZONE)

  1. Go to: https://play.decentraland.zone/index.html?renderer=urn:decentraland:off-chain:renderer-artifacts:feat/add-support-layerless-realm-unity-side&ENV=zone
  2. Test the next features:
  • JumpIn from "Explore & Play".
  • JumpIn from "Friends".
  • Online status in your friends.
  • Users markers in the minimap.

Test realms with the old approach (ORG)

  1. Go to: https://play.decentraland.zone/index.html?renderer=urn:decentraland:off-chain:renderer-artifacts:feat/add-support-layerless-realm-unity-side&ENV=org
  2. Test the next features:
  • JumpIn from "Explore & Play".
  • JumpIn from "Friends".
  • Online status in your friends.
  • Users markers in the minimap.

Our Code Review Standards

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

@sandrade-dcl sandrade-dcl self-assigned this Jul 23, 2021
@github-actions
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@D4rWiNSS D4rWiNSS left a comment

Choose a reason for hiding this comment

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

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Assessment of changes needed for layer removal

4 participants