decentraland / decentraland/unity-explorer
[TECH DEBT] Communities | FRAN & DAVIDE: Fix this, we should be paginating results here.
- 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
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