Comment 4 for bug 1089657

Revision history for this message
Jason Stephenson (jstephenson) wrote :

Dmitry,

Thanks for the patch. I've spent several hours looking at and testing it this morning and it works for me! I did make 1 small change before committing the code. I change the initial value for this.count_for_display from 0 to 1 so that the display would be more consistent with other list interfaces in the staff client. (The offline transaction management interface for instance starts list number for display at 1.)

If you do get accepted as a GSOC student, or if you'd like to make future contributions to Evergreen, please send a public ssh key to the git admins (<email address hidden>). See http://www.evergreen-ils.org/dokuwiki/doku.php?id=dev:git for more information.

The patch has been pushed to master and rel_2_3 for great justice!

Thanks,
Jason Stephenson