search

mohit5189 / SurvayApp

0 stars 0 forks source link

Survey paging is not implemented #6

Open edgarss opened 4 years ago

mohit5189 commented 4 years ago

I didn't get this point, I though paging is just for the case when the developer feels that data is too much. Can you please explain if the pagination feature was expected in the assignment?

mohit5189 commented 4 years ago

Also i just check that this api giving just 4-5 records. If you want i can add code for pagination but that won’t be visible on UI because even if we fetch 5 records per page, it will just return all responses.

edgarss commented 4 years ago

sorry for late reply, that is because the default page size is 5, but there are more records than 5

mohit5189 commented 4 years ago

Sure i will add pagination 👍

mohit5189 commented 4 years ago

add changes for pagination, please check