fix: ab registry manifest fallback case sensitivity + lsd manifest fallback by lorenzo-ranciaffi · Pull Request #7956 · decentraland/unity-explorer

lorenzo-ranciaffi · 2026-03-31T09:24:16Z

Pull Request Description

What does this PR change?

This PR makes sure that AB registry version result is case insensitive: many offchain wearables have capital letters that before weren't matching. E.g.:

urn:decentraland:off-chain:base-avatars:BaseFemale
urn:decentraland:off-chain:base-avatars:BaseMale
urn:decentraland:off-chain:base-avatars:distressed_black_Jeans
urn:decentraland:off-chain:base-avatars:Espadrilles
urn:decentraland:off-chain:base-avatars:Mustache_Short_Beard
urn:decentraland:off-chain:base-avatars:Red_topcoat
urn:decentraland:off-chain:base-avatars:SchoolShoes
urn:decentraland:off-chain:base-avatars:Thunder_earring

Where ever an Emote or Wearable is requested, the AB Registry is used (LSD included) so that the manifest is always present.

On top of that I noticed that when the inventory page had only base emotes, a request to the registry was made with an empty pointers array, causing a 400 log. With an early exit, this is now prevented.

Test Instructions

Test Steps

Login normally, scroll the whole wearable and emote inventory waiting for all pages to load-> verify no AssetBundleManifestFallbackHelper: AB Manifest Fallback requested message is spotted in the log
Start a scene locally and repeat step 1 -> same result of no error logs
Try to both load/kill portable experiences and check the logs -> no error should appear

Quality Checklist

Changes have been tested locally
Documentation has been updated (if required)
Performance impact has been considered
For SDK features: Test scene is included

Code Review Reference

Please review our Code Review Standards before submitting.

…llback

github-actions · 2026-03-31T09:24:32Z

Windows and Mac build successful in Unity Cloud! You can find a link to the downloadable artifact below.

Name	Link
Commit	`1bf20d0`
Logs	https://github.com/decentraland/unity-explorer/actions/runs/24086338341
Download Windows	https://github.com/decentraland/unity-explorer/suites/63625150726/artifacts/6308702123
Download Windows S3	https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/fix/7608-ab-registry-fallback/pr-20767-1bf20d0/Decentraland_windows64.zip
Download Mac	https://github.com/decentraland/unity-explorer/suites/63625150726/artifacts/6308378712
Download Mac S3	https://explorer-artifacts.decentraland.org/@dcl/unity-explorer/branch/fix/7608-ab-registry-fallback/pr-20767-1bf20d0/Decentraland_macos.zip
Built on	2026-04-07T15:05:32Z

claude · 2026-03-31T10:33:50Z

Claude finished @lorenzo-ranciaffi's task in 2m 59s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

The fixes are correct and well-targeted. The case-insensitive dictionary (StringComparer.InvariantCultureIgnoreCase), the AB registry integration in LoadElementsByPointersSystem, and the early-exit for empty pointer arrays all address real bugs cleanly.

Two inline comments posted. One additional finding that couldn't be placed inline:

AssetBundleRegistryVersionHelper.cs (unchanged line ~80): result.versions.Add(element.pointers[0], ...) — if the registry ever returns an entry with an empty pointers array, this throws IndexOutOfRangeException. Pre-existing issue, but easy to guard with if (element.pointers.Length == 0) continue; before the Add.

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

Explorer/Assets/DCL/AvatarRendering/Loading/Systems/LoadElementsByPointersSystem.cs

Explorer/Assets/DCL/AvatarRendering/Emotes/Systems/Load/LoadEmotesByPointersSystem.cs

github-actions · 2026-03-31T10:37:12Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

claude · 2026-03-31T10:45:00Z

Claude finished @lorenzo-ranciaffi's task in 3m 13s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

The core fixes are solid: case-insensitive dictionary for AB registry matching, AB registry integration in LoadElementsByPointersSystem, early-exit for empty pointer arrays, and the GenericPostArguments reuse across both the registry and content calls is safe since CreateJson stores an immutable string.

One finding:

Constructor visibility — Both LoadWearablesDTOByPointersSystem and LoadEmotesByPointersSystem constructors were changed to private, but the code standard requires internal (CLAUDE.md: "Constructors: Marked internal"). LoadWearablesDTOByPointersSystem was previously correct (internal) and regressed; LoadEmotesByPointersSystem was previously wrong (public) but should become internal, not private. Two inline comments posted.

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

Explorer/Assets/DCL/AvatarRendering/Wearables/Systems/Load/LoadWearablesDTOByPointersSystem.cs

Explorer/Assets/DCL/AvatarRendering/Emotes/Systems/Load/LoadEmotesByPointersSystem.cs

github-actions · 2026-03-31T10:48:31Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

claude · 2026-03-31T10:50:17Z

Claude finished @lorenzo-ranciaffi's task in 1m 52s —— View job

Claude finished @lorenzo-ranciaffi's task —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings have been addressed:

✅ Constructor visibility: both LoadWearablesDTOByPointersSystem and LoadEmotesByPointersSystem are now internal
✅ Double space before wearableVersions.windows.buildDate fixed in LoadTrimmedElementsByIntentionSystem.cs
✅ urlsSource duplication removed from LoadEmotesByPointersSystem — now inherited from base class
✅ IndexOutOfRangeException guard added in AssetBundleRegistryVersionHelper.cs (element.pointers.Length == 0 check with LogException)

No new issues found. The CheckAssetBundleManifestFallbackAsync call after the registry check is intentional — it skips the expensive fallback path when the manifest is already set and only runs InjectContent, as documented by the comment. GenericPostArguments reuse is safe (immutable string).

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-03-31T10:52:29Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

…rsions

claude · 2026-03-31T13:33:40Z

Claude finished @lorenzo-ranciaffi's task in 2m 13s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings have been addressed. One minor issue found in this sync:

LoadElementsByPointersSystem.cs line 107 — // //Could be removed... has a double // prefix, a typo from the edit. Inline suggestion posted.

Everything else is correct:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary (case-insensitive URN matching)
✅ EntitiesActiveElements new URL always points to the AB registry (correct — wearable/emote lookups should always use the registry, bypassing the LSD feature flag is intentional here)
✅ GenericPostArguments reuse across both registry and content calls is safe (immutable string)
✅ Early exit for empty pointers array prevents spurious 400 responses
✅ element.pointers.Length == 0 guard with LogException in AssetBundleRegistryVersionHelper
✅ Constructor visibility (internal) correct on both subclasses
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

Explorer/Assets/DCL/AvatarRendering/Loading/Systems/LoadElementsByPointersSystem.cs

github-actions · 2026-03-31T13:36:16Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

claude · 2026-03-31T13:36:54Z

Claude finished @lorenzo-ranciaffi's task in 2m 47s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings have been addressed and no new issues found in this sync:

✅ // // double-prefix comment typo in LoadElementsByPointersSystem.cs is fixed
✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary for case-insensitive URN matching
✅ Early exit for empty pointers array prevents spurious 400 responses
✅ element.pointers.Length == 0 guard with LogException in AssetBundleRegistryVersionHelper
✅ EntitiesActiveElements URL always points to AB registry, bypassing the feature flag (intentional — wearable/emote lookups must always use the registry regardless of the ASSET_BUNDLE_FALLBACK flag)
✅ Constructor visibility (internal) correct on both subclasses
✅ GenericPostArguments reuse across registry + content calls is safe (immutable string)
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, so the new enum case is handled automatically

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-03-31T13:40:16Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

Explorer/Assets/DCL/Infrastructure/ECS/StreamableLoading/AssetBundlesVersions.cs

Explorer/Assets/DCL/NetworkDefinitions/Browser/DecentralandUrlsSource.cs

claude · 2026-04-01T10:47:07Z

Claude finished @lorenzo-ranciaffi's task in 2m 46s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings have been addressed. No new issues found in this sync.

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary (case-insensitive URN matching for off-chain wearables like BaseFemale, SchoolShoes, etc.)
✅ Early exit in GetABRegistryVersionsByPointersAsync when pointers.Count == 0 prevents spurious 400 responses
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ EntitiesActiveElements URL consistently used across BatchPointersSystemBase, ResolveWearablePromisesSystem, and LoadEmotesByPointersSystem — always routes wearable/emote lookups to the AB registry, bypassing the ASSET_BUNDLE_FALLBACK feature flag (intentional per PR description)
✅ Constructor visibility (internal) correct on both LoadWearablesDTOByPointersSystem and LoadEmotesByPointersSystem
✅ GatewayUrlsSource inherits from DecentralandUrlsSource so the new EntitiesActiveElements enum case is handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements
✅ No comment typos, no formatting issues

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-04-01T10:50:13Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

claude · 2026-04-01T11:05:27Z

Claude finished @lorenzo-ranciaffi's task in 3m 5s —— View job

Claude finished reviewing —

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings remain addressed. No new issues found in this sync.

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary for case-insensitive URN matching
✅ Early exit when pointers.Count == 0 prevents spurious 400 responses
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ EntitiesActiveElements URL always routes wearable/emote lookups to the AB registry, bypassing the ASSET_BUNDLE_FALLBACK feature flag (intentional)
✅ Constructor visibility (internal) correct on both LoadWearablesDTOByPointersSystem and LoadEmotesByPointersSystem
✅ Comment double-// typo fixed in LoadElementsByPointersSystem.cs
✅ Double space in LoadTrimmedElementsByIntentionSystem.cs fixed
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, so the new EntitiesActiveElements enum case is handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

claude · 2026-04-01T11:05:38Z

Claude finished @lorenzo-ranciaffi's task in 1m 16s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings have been addressed. No new issues found in this sync.

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary (case-insensitive URN matching for off-chain wearables like BaseFemale, SchoolShoes, etc.)
✅ Early exit in GetABRegistryVersionsByPointersAsync when pointers.Count == 0 prevents spurious 400 responses — placed before STRING_BUILDER_POOL acquisition, correctly before any async work
✅ UniTask.SwitchToMainThread() correctly moved to the second overload (the one that makes the actual web request), not needed in the first overload which only does string construction
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ EntitiesActiveElements URL consistently used across BatchPointersSystemBase, ResolveWearablePromisesSystem, and LoadEmotesByPointersSystem — always routes wearable/emote lookups to the AB registry
✅ Constructor visibility (internal) correct on LoadEmotesByPointersSystem
✅ AssetBundlesVersions.versions changed from readonly field to private set property — semantically equivalent, no regression
✅ GenericPostArguments reuse across registry + content calls is safe (immutable string)
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, new EntitiesActiveElements case handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-04-01T11:07:17Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

github-actions · 2026-04-01T11:08:56Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

AnsisMalins · 2026-04-01T11:28:51Z

PR 7956

PR number: 7956
Windows change build
Windows baseline build
macOS change build
macOS baseline build

Intel Core i3

Change samples: 2670
Baseline samples: 2675

Metric	Change	Baseline	Delta	Improvement
CPU average	33.7 ms	33.6 ms	0.1 ms	-0.2%	⚪
CPU 1% worst	59.5 ms	53.6 ms	5.9 ms	-11.0%	🔴
CPU 0.1% worst	127.6 ms	138.1 ms	-10.5 ms	7.6%	🟢
GPU average	15.3 ms	16.1 ms	-0.8 ms	5.3%	🟢
GPU 1% worst	21.9 ms	21.9 ms	-0.0 ms	0.1%	⚪
GPU 0.1% worst	27.3 ms	25.5 ms	1.9 ms	-7.3%	🔴

AnsisMalins · 2026-04-01T14:14:30Z

PR 7956

PR number: 7956
Windows change build
Windows baseline build
macOS change build
macOS baseline build

Intel Core i3

Change samples: 2672
Baseline samples: 2671

Metric	Change	Baseline	Delta	Improvement
CPU average	33.6 ms	33.7 ms	-0.0 ms	0.1%	⚪
CPU 1% worst	54.8 ms	59.9 ms	-5.1 ms	8.6%	🟢
CPU 0.1% worst	120.1 ms	147.1 ms	-27.0 ms	18.3%	🟢
GPU average	15.7 ms	15.7 ms	0.0 ms	-0.2%	⚪
GPU 1% worst	20.8 ms	21.6 ms	-0.9 ms	4.0%	🟢
GPU 0.1% worst	22.6 ms	28.6 ms	-6.0 ms	21.1%	🟢

AnsisMalins · 2026-04-02T11:48:15Z

PR 7956

PR number: 7956
Windows change build
Windows baseline build
macOS change build
macOS baseline build

Intel Core i3

Change samples: 2668
Baseline samples: 2672

Metric	Change	Baseline	Delta	Improvement
CPU average	33.7 ms	33.6 ms	0.0 ms	-0.1%	⚪
CPU 1% worst	55.2 ms	54.4 ms	0.8 ms	-1.4%	⚪
CPU 0.1% worst	130.0 ms	119.7 ms	10.3 ms	-8.6%	🔴
GPU average	15.4 ms	15.8 ms	-0.5 ms	3.1%	🟢
GPU 1% worst	22.5 ms	20.9 ms	1.5 ms	-7.3%	🔴
GPU 0.1% worst	24.4 ms	22.7 ms	1.7 ms	-7.7%	🔴

claude · 2026-04-02T13:31:30Z

Claude finished @lorenzo-ranciaffi's task in 1m 20s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings remain addressed. No new issues found in this sync (merge of dev into the PR branch).

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary for case-insensitive URN matching (BaseFemale, SchoolShoes, etc.)
✅ Early exit when pointers.Count == 0 prevents spurious 400 responses — placed before STRING_BUILDER_POOL acquisition
✅ GetABRegistryVersionsByPointersAsync split into two overloads correctly — UniTask.SwitchToMainThread() only in the second overload that makes the web request
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ EntitiesActiveElements URL always routes wearable/emote lookups to AB registry (bypasses ASSET_BUNDLE_FALLBACK feature flag — intentional)
✅ Constructor visibility (internal) correct on LoadEmotesByPointersSystem
✅ AssetBundlesVersions.versions private set property — Create() factory is the only setter, semantically safe
✅ GenericPostArguments reuse across registry + content calls is safe (immutable string)
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, new EntitiesActiveElements case handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper
✅ Comment style and double-space formatting fixes

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-04-02T13:33:12Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

AnsisMalins · 2026-04-03T15:18:04Z

PR #7956, run #23907720450

Builds: Windows change, Windows baseline, macOS change, macOS baseline

Intel Core i3

Metric	Change	Baseline	Delta	Improvement
Samples	2655	2672
CPU average	33.8 ms	33.6 ms	0.2 ms	-0.6%	⚪
CPU 1% worst	68.1 ms	55.8 ms	12.3 ms	-22.0%	🔴
CPU 0.1% worst	251.4 ms	132.1 ms	119.3 ms	-90.3%	🔴
GPU average	16.2 ms	15.4 ms	0.8 ms	-5.2%	🔴
GPU 1% worst	23.6 ms	21.0 ms	2.5 ms	-12.1%	🔴
GPU 0.1% worst	28.6 ms	23.2 ms	5.4 ms	-23.5%	🔴

claude · 2026-04-07T12:50:32Z

Claude finished @lorenzo-ranciaffi's task in 1m 51s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings remain addressed. No new issues found in this sync.

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary for case-insensitive URN matching (BaseFemale, SchoolShoes, etc.)
✅ Early exit when pointers.Count == 0 prevents spurious 400 responses — placed before STRING_BUILDER_POOL acquisition, correctly before any async work
✅ GetABRegistryVersionsByPointersAsync split into two overloads — UniTask.SwitchToMainThread() correctly in the second overload only (the one that makes the web request)
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ AssetBundlesVersions struct object initializer: private set is accessible within Create() (same type) — valid C# 10 pattern
✅ EntitiesActiveElements URL always routes wearable/emote lookups to the AB registry, bypassing the ASSET_BUNDLE_FALLBACK feature flag (intentional)
✅ Constructor visibility (internal) correct on LoadEmotesByPointersSystem
✅ GenericPostArguments reuse across registry + content calls is safe (immutable string)
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, new EntitiesActiveElements case handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper
✅ Comment formatting and double-space fixes

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-04-07T12:52:48Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

claude · 2026-04-07T14:20:16Z

Claude finished @lorenzo-ranciaffi's task in 2m 41s —— View job

PR Review

Gathering context
Reading changed files
Analyzing code and posting review

All previous review findings remain addressed. No new issues found in this sync.

Verified clean:

✅ StringComparer.InvariantCultureIgnoreCase on the versions dictionary for case-insensitive URN matching (BaseFemale, SchoolShoes, etc.)
✅ Early exit when pointers.Count == 0 prevents spurious 400 responses — correctly placed before STRING_BUILDER_POOL acquisition
✅ GetABRegistryVersionsByPointersAsync split into two overloads — bodyBuilder.ToString() creates an immutable string copy before sbScope is disposed, so GenericPostArguments.CreateJson holds a valid reference throughout the second overload's lifetime
✅ UniTask.SwitchToMainThread() correctly in the second overload only (the one that calls URLBuilder and makes the web request)
✅ element.pointers.Length == 0 guard with ReportHub.LogException in AssetBundleRegistryVersionHelper
✅ CreateAssetBundleManifestFallbackAsync (new) correctly skips null/empty check and Sentry breadcrumb for scenes fetched from catalyst — the skip of InjectContent for the LSD path is intentional (local assets don't need manifest content injection)
✅ CheckAssetBundleManifestFallbackAsync(EntityDefinitionBase) retains the InjectContent call after the internal check — assetBundleManifestVersion is guaranteed non-null after CheckAssetBundleManifestFallbackInternalAsync (either set to CreateManualManifest, CreateFromFallback, or CreateFailed, or was already non-null/non-empty)
✅ EntitiesActiveElements URL always routes wearable/emote lookups to the AB registry, bypassing the ASSET_BUNDLE_FALLBACK feature flag (intentional)
✅ Constructor visibility (internal) correct on LoadEmotesByPointersSystem
✅ AssetBundlesVersions.versions private set property — Create() factory is the only setter, semantically safe
✅ GatewayUrlsSource inherits from DecentralandUrlsSource, new EntitiesActiveElements case handled automatically
✅ Test in BatchPointersSystemShould.cs updated to stub DecentralandUrl.EntitiesActiveElements
✅ Unused using Sentry and using UnityEngine removed from AssetBundleManifestFallbackHelper
✅ Comment formatting and double-space fixes applied

REVIEW_RESULT: PASS ✅
COMPLEXITY: COMPLEX
QA_REQUIRED: YES

github-actions · 2026-04-07T14:23:26Z

🔍 Claude reviewed this PR and found no blocking issues, but assessed it as complex — human DEV review is still required before merging.

fix: ab registry manifest fallback case sensitivity + lsd manifest fa…

6f69335

…llback

lorenzo-ranciaffi self-assigned this Mar 31, 2026

lorenzo-ranciaffi added this to Explorer Alpha Mar 31, 2026

github-project-automation bot moved this to Todo in Explorer Alpha Mar 31, 2026

accept both json payload and urn collection

47985f6

lorenzo-ranciaffi marked this pull request as ready for review March 31, 2026 10:33

lorenzo-ranciaffi requested review from a team as code owners March 31, 2026 10:33

github-actions bot requested a review from anicalbano March 31, 2026 10:33

lorenzo-ranciaffi moved this from Todo to With QA / Awaiting Review in Explorer Alpha Mar 31, 2026

claude bot reviewed Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/AvatarRendering/Loading/Systems/LoadElementsByPointersSystem.cs Outdated Show resolved Hide resolved

claude bot reviewed Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/AvatarRendering/Emotes/Systems/Load/LoadEmotesByPointersSystem.cs Outdated Show resolved Hide resolved

review changes

815571c

claude bot reviewed Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/AvatarRendering/Wearables/Systems/Load/LoadWearablesDTOByPointersSystem.cs Outdated Show resolved Hide resolved

claude bot reviewed Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/AvatarRendering/Emotes/Systems/Load/LoadEmotesByPointersSystem.cs Outdated Show resolved Hide resolved

suggested changes

63a62b2

use registry for all wearables/emotes instead of using it only for ve…

6bc3a6c

…rsions

fix typo

220affe

claude bot reviewed Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/AvatarRendering/Loading/Systems/LoadElementsByPointersSystem.cs Outdated Show resolved Hide resolved

mihakrajnc approved these changes Mar 31, 2026

View reviewed changes

Explorer/Assets/DCL/Infrastructure/ECS/StreamableLoading/AssetBundlesVersions.cs Outdated Show resolved Hide resolved

Explorer/Assets/DCL/NetworkDefinitions/Browser/DecentralandUrlsSource.cs Show resolved Hide resolved

github-project-automation bot moved this from With QA / Awaiting Review to In Progress in Explorer Alpha Mar 31, 2026

added comment

5f3e971

dalkia approved these changes Apr 1, 2026

View reviewed changes

lorenzo-ranciaffi and others added 2 commits April 1, 2026 13:04

removed weird shenanigans

aa22d2f

Merge branch 'dev' into fix/7608-ab-registry-fallback

5b89997

lorenzo-ranciaffi enabled auto-merge (squash) April 1, 2026 13:02

Merge branch 'dev' into fix/7608-ab-registry-fallback

813a551

Merge branch 'dev' into fix/7608-ab-registry-fallback

9ff6a4f

lorenzo-ranciaffi requested a review from dalkia April 7, 2026 13:53

always download manifest with no exception for px

1bf20d0

dalkia approved these changes Apr 7, 2026

View reviewed changes

Conversation

lorenzo-ranciaffi commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Description

What does this PR change?

Test Instructions

Test Steps

Quality Checklist

Code Review Reference

Uh oh!

github-actions bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 31, 2026

Uh oh!

claude bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 31, 2026

Uh oh!

claude bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

github-actions bot commented Mar 31, 2026

Uh oh!

claude bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

Uh oh!

github-actions bot commented Mar 31, 2026

Uh oh!

claude bot commented Mar 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

github-actions bot commented Mar 31, 2026

Uh oh!

Uh oh!

Uh oh!

claude bot commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

github-actions bot commented Apr 1, 2026

Uh oh!

claude bot commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

claude bot commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Review

Uh oh!

github-actions bot commented Apr 1, 2026

Uh oh!

github-actions bot commented Apr 1, 2026

Uh oh!

AnsisMalins commented Apr 1, 2026

PR 7956

Intel Core i3

Uh oh!

AnsisMalins commented Apr 1, 2026

PR 7956

Intel Core i3

Uh oh!

AnsisMalins commented Apr 2, 2026

PR 7956

Intel Core i3

Uh oh!

lorenzo-ranciaffi commented Mar 31, 2026 •

edited

Loading

github-actions bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Mar 31, 2026 •

edited

Loading

claude bot commented Apr 1, 2026 •

edited

Loading

claude bot commented Apr 1, 2026 •

edited

Loading

claude bot commented Apr 1, 2026 •

edited

Loading

claude bot commented Apr 2, 2026 •

edited

Loading

claude bot commented Apr 7, 2026 •

edited

Loading

claude bot commented Apr 7, 2026 •

edited

Loading