checklist view template limited to 100 items

[k2soule]

Question; what is causing the limitation of checklist view to show only the first 100 items in the folder? Is it something that we can change?

[alecpm]

Currently, the template sets batch=True in the arguments to getFolderContents, which batches the results (default batch size 100). We can remove batching to ensure all results are shown, or add the batch navigation macros to the page so that you can easily navigate between batches of 100 (like on folder_contents and other pages). Which would be preferred?

[k2soule]

My first instinct is to enable the batch navigation macros, so that we don't have to worry about an exhibition checklist with 400 object thumbnails trying to load all at once.

Tom, any objections?

-Kristen

On Mon, Jun 6, 2016 at 7:00 PM, Alec Mitchell notifications@github.com wrote:

Currently, the template sets batch=True in the arguments to getFolderContents, which batches the results (default batch size 100). We can remove batching to ensure all results are shown, or add the batch navigation macros to the page so that you can easily navigate between batches of 100 (like on folder_contents and other pages). Which would be preferred?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/isawnyu/isaw.web/issues/64#issuecomment-224114669, or mute the thread https://github.com/notifications/unsubscribe/AHhIkNrkJZQ8T0CVkHL2KpcI2LyAXPz_ks5qJKaegaJpZM4GXuJC .

[k2soule]

Yes that's my inclination as well. Let's make it so.

On Tuesday, June 7, 2016, Kristen Soule kristen.soule@nyu.edu wrote:

My first instinct is to enable the batch navigation macros, so that we don't have to worry about an exhibition checklist with 400 object thumbnails trying to load all at once.

Tom, any objections?

-Kristen

On Mon, Jun 6, 2016 at 7:00 PM, Alec Mitchell <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

Currently, the template sets batch=True in the arguments to getFolderContents, which batches the results (default batch size 100). We can remove batching to ensure all results are shown, or add the batch navigation macros to the page so that you can easily navigate between batches of 100 (like on folder_contents and other pages). Which would be preferred?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/isawnyu/isaw.web/issues/64#issuecomment-224114669, or mute the thread https://github.com/notifications/unsubscribe/AHhIkNrkJZQ8T0CVkHL2KpcI2LyAXPz_ks5qJKaegaJpZM4GXuJC .

Tom Elliott, Ph.D. Associate Director for Digital Programs and Senior Research Scholar Institute for the Study of the Ancient World (NYU) http://isaw.nyu.edu/people/staff/tom-elliott

[alecpm]

This is deployed to staging. See here for an example:

http://isaw4-dev.atlantides.org/exhibitions/aesthetics/checklist/

[k2soule]

Nice - that has the desired effect. Thanks!

On Tue, Jun 7, 2016 at 2:32 PM, Alec Mitchell notifications@github.com wrote:

This is deployed to staging. See here for an example:

http://isaw4-dev.atlantides.org/exhibitions/aesthetics/checklist/

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/isawnyu/isaw.web/issues/64#issuecomment-224372606, or mute the thread https://github.com/notifications/unsubscribe/AHhIkBLehU0nEuPB6DtWw0wYWREB0PFZks5qJblZgaJpZM4GXuJC .

[paregorios]

thanks, please deploy to production at your leisure

[alecpm]

Done: http://isaw.nyu.edu/exhibitions/aesthetics/checklist/