decentraland / decentraland/unity-explorer
[TECH DEBT] AvatarRendering | it may occur that the requested emote does not support the body shape
- Dominant language
- C#
- Stars
- 23
- Forks
- 17
- Avg merge
- 2d 16h
- Merged PRs (30d)
- 101
Description
### π **Priority Level:**
Low
### π **Area/Component:**
`AvatarRendering\Emotes\Systems\Load\LoadEmotesByPointersSystem.cs`
### π **Description:**
Unresolved TODO comment in the codebase:
> TODO: it may occur that the requested emote does not support the body shape
### π **Current State:**
The following line is present in `AvatarRendering\Emotes\Systems\Load\LoadEmotesByPointersSystem.cs`:
```csharp
// TODO: it may occur that the requested emote does not support the body shape
```
### π‘ **Proposed Solution:**
Address the TODO as described in the comment, or remove it if no longer relevant.
### βοΈ **Impact Assessment:**
#### Performance Impact:
To be assessed when the TODO is picked up.
#### Maintainability Impact:
Unresolved TODOs accumulate as tech debt and obscure the intended final state of the code.
#### Risk of Refactoring:
Low β scoped to the referenced location.
### π **Effort Estimate:**
XS
### π **Dependencies:**
N/A
### π₯οΈ **Additional Notes:**
Created automatically from a bulk TODO audit (todo-list.txt entry #24).
Contributor guide
Research direction
Start by reading AvatarRendering\Emotes\Systems\Load\LoadEmotesByPointersSystem.cs around the TODO and trace how requested emotes and body shapes are loaded. Determine the expected behavior when an emote lacks support for the selected body shape, then verify that the TODO is resolved through the relevant loading behavior or removed if it is no longer applicable.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp, unity
- Domain
- computer-graphics, game-dev
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 52/100