Code review comment for lp:~james-w/summit/multiple-sprints

Revision history for this message
Michael Hall (mhall119) wrote :

17:09 < mhall119> james_w: my only concern about the multi-sprint is that once you
                  add a Sprint record, you lose the default which uses the summit name
17:09 < mhall119> this may cause confusion
17:09 < james_w> hmm
17:09 < mhall119> we'll just have to make sure people know about it
17:09 < james_w> ok
17:10 < mhall119> so right now we'll have uds-p spring
17:10 < mhall119> sprint
17:10 < james_w> I would have put some text on the admin page, but I wasn't sure how
17:10 < mhall119> and if Marianna adds the Linaro sprint, suddenly it won't check the
                  uds-p sprint anymore
17:11 < mhall119> I think just making them aware will suffice for this cycle, but we
                  might want to discuss how to keep it consistent going forward
17:11 < mhall119> either dropping the use of a default, or adding the default as a
                  sprint on initial save of a Summit object
17:12 < james_w> ah, that's a good idea
17:12 < james_w> I'll set this one up, so it won't be a problem unless anyone changes
                 them

« Back to merge proposal