code4romania / code4romania/monitorizare-vot-votanti-api

Pagination doesn't work properly for pending incidents

Open
#37 1 comment 0 reactions 0 assignees View on GitHub
api bug help wanted laravel
Dominant language
PHP
Stars
8
Forks
10
PR merge metrics
No merged PRs in 30d

Description

Related to #8

First two pending incidents pages for /incidents (page 0 and page 1) return the exact same result.

https://portal-votanti-uat.azurewebsites.net/api/incidents?limit=20&page=1&status=Pending
https://portal-votanti-uat.azurewebsites.net/api/incidents?limit=20&page=0&status=Pending

Contributor guide

Open the contributing guide

Research direction

Start at the API handler for /incidents and reproduce the two linked requests with status=Pending, page=0, and page=1. Trace how pagination is applied to pending incidents and verify that the two pages return different results; done means the reported duplication no longer occurs.

Written by the indexing model from the issue text.

Assessment

Tech stack
laravel, php
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.