decentraland / decentraland/unity-explorer

[TECH DEBT] Communities | FRAN & DAVIDE: Fix this, we should be paginating results here.

Open
#9,139 2 comments 0 reactions 0 assignees View on GitHub
3-low performance refactor stale tech debt
Dominant language
C#
Stars
23
Forks
17
Avg merge
2d 16h
Merged PRs (30d)
101

Description

### πŸ“Š **Priority Level:**
Low

### πŸ“ **Area/Component:**
`Communities\CommunitiesDataProvider\CommunitiesDataProvider.cs`

### πŸ“ **Description:**
Unresolved TODO comment in the codebase:

> TODO FRAN & DAVIDE: Fix this, we should be paginating results here.

### πŸ” **Current State:**
The following line is present in `Communities\CommunitiesDataProvider\CommunitiesDataProvider.cs`:

```csharp
if (isStreaming) //TODO FRAN & DAVIDE: Fix this, we should be paginating results here.
```

### πŸ’‘ **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 #16).

Contributor guide

Open the contributing guide

Research direction

Start in Communities\CommunitiesDataProvider\CommunitiesDataProvider.cs at the isStreaming branch and trace how its results are retrieved and consumed. Determine the existing pagination contract and how pagination should apply there; the work is done when streaming results are paginated as intended and the TODO is resolved or removed.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Needs clarification
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.