GET Notifications limit

apavlidi / IT_API

The Internet services of the IT department of Alexander Technological Education Institute of Thessaloniki

http://api.it.teithe.gr/

MIT License

14 stars 5 forks source link

GET Notifications limit #85

Open iamraf opened 5 years ago

iamraf commented 5 years ago

I currently have more than 700 notifications.

I would suggest that we should have a limit of 50 as well as descending sort option to get the latest ones. Or maybe remove old ones and keep some period of 3 months.

apavlidi commented 5 years ago

@h01d Nice catch. You are right we should limit the notificatioins retrieved

asidirop commented 5 years ago

The api should support pagination and the number of elements should be an argument.

iamraf commented 5 years ago

@asidirop announcements do support pagination, I don't think notifications should have.