groton-school / course-planning-tool

Course planning tool to distribute individual Google Sheet course plans to students and advisors based on Blackbaud registration data
GNU General Public License v3.0
0 stars 0 forks source link

figure out versioning for including restructure options #91

Closed github-actions[bot] closed 11 months ago

github-actions[bot] commented 1 year ago

Pragmatically, we don't need these options, since all plans are now at

v0.2.1 (which introduced student folders). It would be slick to be

able to conditionally hide this option when not needed (and to have a

structure for doing this for other restructuring that is version-

based.

https://api.github.com/groton-school/course-planning-tool/blob/01e902bc9c4f584e759ee79be9680280df744e3c/src/AddOn.ts#L29


      Workflow.UpdateCourseList.pickPlan()
    )
    .addSeparator()
    /*
     * TODO figure out versioning for including restructure options
     *   Pragmatically, we don't need these options, since all plans are now at
     *   v0.2.1 (which introduced student folders). It would be slick to be
     *   able to conditionally hide this option when not needed (and to have a
     *   structure for doing this for other restructuring that is version-
     *   based.
     */
    .addItem(
      'Create a single missing student folder…',
      Workflow.Restructure.pickStudentMissingFolder()
github-actions[bot] commented 1 year ago

Closed in 66475203c74210d8f782ba3d352992346aeee72e

battis commented 11 months ago

This would be elegant and cool. But, I think, fundamentally unnecessary, since once all of the course plans are updated, their version becomes moot -- and once students graduate, updating their plans is also moot.