Once #578592: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin and #581652: Add a {signup_log}.count_towards_limit column and adjust limit logic accordingly land, we're going to want to alter that summary form from signup_status_limit to display the right info about the limit type on the node, and if configured for per-status limits, info about the current limits...
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | Signup details.png | 9.82 KB | roball |
| #2 | 583154-1.signup_status_limit_summary_alter.patch | 2.63 KB | dww |
| #2 | 583154-1.signup_status_limit_summary_alter.status.png | 73.14 KB | dww |
| #1 | 583154-1.signup_status_limit_summary_alter.total_.png | 70.97 KB | dww |
Comments
Comment #1
dwwThis patch depends on #578592-2: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin, but I think this looks pretty nice for both global vs. per-status signup limits.
Comment #2
dwwSorry, here's the other screenshot (showing how it looks when the node is configured for per-status limits) and the patch...
Comment #3
mlsamuelson commentedI've tested the patch, after applying #578592: Cleanup + fix the "Signup summary" at the top of node/N/signups/admin and it works as advertised. I don't see any problems with it. :)
Comment #4
dwwCommitted to HEAD. Thanks!
Comment #5
roball commentedHm, I don't have the columns "Status" and "Operations", as shown in the attached screenshot. Am I missing something?
Comment #6
dww@roball: Please read the README.txt file included with signup_status. In particular, the "Setup" section. That's what you're missing.
Comment #7
roball commentedProblem solved as discussed in #589030: No 'Edit signup' link on the node/N/signups/admin tab. Thanks for your help.