search

dnnsoftware / Dnn.Platform

DNN (formerly DotNetNuke) is the leading open source web content management platform (CMS) in the Microsoft ecosystem.
https://dnncommunity.org/
MIT License
1.01k stars 745 forks source link

Add MaximumPageSize const property for limit page size #6034

Closed Mostafa-Moafi closed 1 month ago

Mostafa-Moafi commented 1 month ago

Add MaximumPageSize const property for limit page size Fixes #6033

valadas commented 1 month ago

Just to be sure this maximum is only per page right, one could still obtain more results but on other pages correct?

Mostafa-Moafi commented 1 month ago

@valadas yes, of course. I just set the maximum page size per page.

valadas commented 1 month ago

Ok, I am not too concerned with this but like Brian said, it would be nicer if that value comes from the web.config or a host setting and could be changed without having to recompile. I could see potential implementations where that API gets called from 3rd party modules that may need more results and not support paging (would only get page 1).

Mostafa-Moafi commented 1 month ago

@bdukes @valadas OK, I add a new input to the search section in the persona bar. do you agree?

valadas commented 1 month ago

Sounds like a decent place to put that yes.

Mostafa-Moafi commented 1 month ago

@valadas Thanks. I added a new field in the search section.