From 0bad562c90f95c03221461d1824ea7a94aa9e04c Mon Sep 17 00:00:00 2001
From: Michael Chadwick <michael.chadwick@gmail.com>
Date: Fri, 4 Oct 2024 10:40:35 -0700
Subject: [PATCH 1/4] removed did-insert from new course component

---
 .../frontend/app/components/courses/new.hbs   | 42 ++++++++++---------
 .../frontend/app/components/courses/new.js    | 25 ++++++-----
 2 files changed, 36 insertions(+), 31 deletions(-)

diff --git a/packages/frontend/app/components/courses/new.hbs b/packages/frontend/app/components/courses/new.hbs
index d5ea2e8df8..adb826f9be 100644
--- a/packages/frontend/app/components/courses/new.hbs
+++ b/packages/frontend/app/components/courses/new.hbs
@@ -1,5 +1,5 @@
 {{#let (unique-id) as |templateId|}}
-  <div class="courses-new" {{did-insert (perform this.load)}} data-test-courses-new>
+  <div class="courses-new" data-test-courses-new>
     {{#unless this.load.isRunning}}
       <h4>
         {{t "general.newCourse"}}
@@ -28,26 +28,28 @@
           <label for="year-{{templateId}}">
             {{t "general.academicYear"}}:
           </label>
-          <select
-            id="year-{{templateId}}"
-            data-test-year
-            {{on "change" (pick "target.value" this.setYear)}}
-          >
-            <option disabled="" selected={{is-empty this.selectedYear}} value="">
-              {{t "general.selectAcademicYear"}}
-            </option>
-            {{#each this.years as |year|}}
-              <option selected={{eq year this.selectedYear}} value={{year}}>
-                {{#if this.academicYearCrossesCalendarYearBoundaries}}
-                  {{year}}
-                  -
-                  {{add year 1}}
-                {{else}}
-                  {{year}}
-                {{/if}}
+          {{#if this.academicYearData.isResolved}}
+            <select
+              id="year-{{templateId}}"
+              data-test-year
+              {{on "change" (pick "target.value" this.setYear)}}
+            >
+              <option disabled="" selected={{is-empty this.selectedYear}} value="">
+                {{t "general.selectAcademicYear"}}
               </option>
-            {{/each}}
-          </select>
+              {{#each this.years as |year|}}
+                <option selected={{eq year this.selectedYear}} value={{year}}>
+                  {{#if this.academicYearCrossesCalendarYearBoundaries}}
+                    {{year}}
+                    -
+                    {{add year 1}}
+                  {{else}}
+                    {{year}}
+                  {{/if}}
+                </option>
+              {{/each}}
+            </select>
+          {{/if}}
         </div>
         <div class="buttons">
           <button
diff --git a/packages/frontend/app/components/courses/new.js b/packages/frontend/app/components/courses/new.js
index e8a1de2b7a..aa465365ef 100644
--- a/packages/frontend/app/components/courses/new.js
+++ b/packages/frontend/app/components/courses/new.js
@@ -1,10 +1,11 @@
 import Component from '@glimmer/component';
-import { tracked } from '@glimmer/tracking';
+import { cached, tracked } from '@glimmer/tracking';
 import { action } from '@ember/object';
 import { service } from '@ember/service';
 import { DateTime } from 'luxon';
 import { validatable, Length, NotBlank } from 'ilios-common/decorators/validation';
-import { dropTask, restartableTask } from 'ember-concurrency';
+import { dropTask } from 'ember-concurrency';
+import { TrackedAsyncData } from 'ember-async-data';
 
 @validatable
 export default class CoursesNewComponent extends Component {
@@ -21,6 +22,17 @@ export default class CoursesNewComponent extends Component {
     super(...arguments);
     const thisYear = DateTime.now().year;
     this.years = [thisYear - 2, thisYear - 1, thisYear, thisYear + 1, thisYear + 2];
+
+    if (this.args.currentYear && this.years.includes(parseInt(this.args.currentYear.id, 10))) {
+      this.setYear(this.args.currentYear.id);
+    }
+  }
+
+  @cached
+  get academicYearData() {
+    return new TrackedAsyncData(
+      this.iliosConfig.itemFromConfig('academicYearCrossesCalendarYearBoundaries'),
+    );
   }
 
   @action
@@ -45,15 +57,6 @@ export default class CoursesNewComponent extends Component {
     }
   }
 
-  load = restartableTask(async () => {
-    this.academicYearCrossesCalendarYearBoundaries = await this.iliosConfig.itemFromConfig(
-      'academicYearCrossesCalendarYearBoundaries',
-    );
-    if (this.args.currentYear && this.years.includes(parseInt(this.args.currentYear.id, 10))) {
-      this.setYear(this.args.currentYear.id);
-    }
-  });
-
   saveCourse = dropTask(async () => {
     this.addErrorDisplayFor('title');
     const isValid = await this.isValid();

From bcc6cf5f565fe41b69eece3ea641cfe5f9a74035 Mon Sep 17 00:00:00 2001
From: Michael Chadwick <michael.chadwick@gmail.com>
Date: Fri, 4 Oct 2024 10:46:48 -0700
Subject: [PATCH 2/4] update lint-todo

---
 packages/frontend/.lint-todo | 1 +
 1 file changed, 1 insertion(+)

diff --git a/packages/frontend/.lint-todo b/packages/frontend/.lint-todo
index af7c8ddb86..0871119dbf 100644
--- a/packages/frontend/.lint-todo
+++ b/packages/frontend/.lint-todo
@@ -62,3 +62,4 @@ add|ember-template-lint|no-at-ember-render-modifiers|10|6|10|6|d919d2af254f782c0
 add|ember-template-lint|no-at-ember-render-modifiers|11|6|11|6|940005188b476a060b0e5d3f05baea24ba178878|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/session-type.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|10|6|10|6|d919d2af254f782c01fe2ba15416673e52e91124|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/term.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|11|6|11|6|940005188b476a060b0e5d3f05baea24ba178878|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/term.hbs
+remove|ember-template-lint|no-at-ember-render-modifiers|4|4|4|4|23cd787c79c34a628dadb6e96dd4004d42eebb79|1722902400000|1730682000000|1754006400000|app/components/courses/new.hbs

From ef357510aaad09235109d3c57d788a592d7d7e88 Mon Sep 17 00:00:00 2001
From: Michael Chadwick <michael.chadwick@gmail.com>
Date: Fri, 4 Oct 2024 12:02:31 -0700
Subject: [PATCH 3/4] add another layer of async loading

---
 packages/frontend/app/components/courses/new.hbs | 2 +-
 packages/frontend/app/components/courses/new.js  | 9 +++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/packages/frontend/app/components/courses/new.hbs b/packages/frontend/app/components/courses/new.hbs
index adb826f9be..8fed7bca5d 100644
--- a/packages/frontend/app/components/courses/new.hbs
+++ b/packages/frontend/app/components/courses/new.hbs
@@ -28,7 +28,7 @@
           <label for="year-{{templateId}}">
             {{t "general.academicYear"}}:
           </label>
-          {{#if this.academicYearData.isResolved}}
+          {{#if this.academicYearCrossesCalendarYearBoundariesData.isResolved}}
             <select
               id="year-{{templateId}}"
               data-test-year
diff --git a/packages/frontend/app/components/courses/new.js b/packages/frontend/app/components/courses/new.js
index aa465365ef..0a465941ec 100644
--- a/packages/frontend/app/components/courses/new.js
+++ b/packages/frontend/app/components/courses/new.js
@@ -15,7 +15,6 @@ export default class CoursesNewComponent extends Component {
 
   @tracked @NotBlank() selectedYear;
   @tracked @NotBlank() @Length(3, 200) title;
-  @tracked academicYearCrossesCalendarYearBoundaries;
   @tracked years;
 
   constructor() {
@@ -29,12 +28,18 @@ export default class CoursesNewComponent extends Component {
   }
 
   @cached
-  get academicYearData() {
+  get academicYearCrossesCalendarYearBoundariesData() {
     return new TrackedAsyncData(
       this.iliosConfig.itemFromConfig('academicYearCrossesCalendarYearBoundaries'),
     );
   }
 
+  get academicYearCrossesCalendarYearBoundaries() {
+    return this.academicYearCrossesCalendarYearBoundariesData.isResolved
+      ? this.academicYearCrossesCalendarYearBoundariesData.value
+      : false;
+  }
+
   @action
   setYear(year) {
     this.selectedYear = parseInt(year, 10);

From b6cc5c6cf4d15f7ecee957051dd8afd02e4ee5c5 Mon Sep 17 00:00:00 2001
From: Michael Chadwick <michael.chadwick@gmail.com>
Date: Fri, 4 Oct 2024 12:39:56 -0700
Subject: [PATCH 4/4] updated lint-todo list properly

---
 packages/frontend/.lint-todo | 2 --
 1 file changed, 2 deletions(-)

diff --git a/packages/frontend/.lint-todo b/packages/frontend/.lint-todo
index 0871119dbf..6f9ae24690 100644
--- a/packages/frontend/.lint-todo
+++ b/packages/frontend/.lint-todo
@@ -7,7 +7,6 @@ add|ember-template-lint|no-at-ember-render-modifiers|4|2|4|2|23cd787c79c34a628da
 add|ember-template-lint|no-at-ember-render-modifiers|5|2|5|2|e5120f87b74c5ae8e4c76b9089e0b4a4504c6e3c|1722902400000|1730682000000|1754006400000|app/components/user-profile-roles.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|4|2|4|2|1fb0566922ce4f066916e5e2931f650f69d7cfba|1722902400000|1730682000000|1754006400000|app/components/visualizer-program-year-objectives.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|5|2|5|2|38e65b45b56fdfd4160d3b0884114b6643e3a036|1722902400000|1730682000000|1754006400000|app/components/visualizer-program-year-objectives.hbs
-add|ember-template-lint|no-at-ember-render-modifiers|4|4|4|4|23cd787c79c34a628dadb6e96dd4004d42eebb79|1722902400000|1730682000000|1754006400000|app/components/courses/new.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|5|2|5|2|23cd787c79c34a628dadb6e96dd4004d42eebb79|1722902400000|1730682000000|1754006400000|app/components/curriculum-inventory/new-report.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|4|2|4|2|23cd787c79c34a628dadb6e96dd4004d42eebb79|1722902400000|1730682000000|1754006400000|app/components/curriculum-inventory/new-sequence-block.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|5|2|5|2|77e3831e4ae1b00caee1f808711f2e26ab452a23|1722902400000|1730682000000|1754006400000|app/components/curriculum-inventory/new-sequence-block.hbs
@@ -62,4 +61,3 @@ add|ember-template-lint|no-at-ember-render-modifiers|10|6|10|6|d919d2af254f782c0
 add|ember-template-lint|no-at-ember-render-modifiers|11|6|11|6|940005188b476a060b0e5d3f05baea24ba178878|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/session-type.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|10|6|10|6|d919d2af254f782c01fe2ba15416673e52e91124|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/term.hbs
 add|ember-template-lint|no-at-ember-render-modifiers|11|6|11|6|940005188b476a060b0e5d3f05baea24ba178878|1722902400000|1730682000000|1754006400000|app/components/reports/subject/new/term.hbs
-remove|ember-template-lint|no-at-ember-render-modifiers|4|4|4|4|23cd787c79c34a628dadb6e96dd4004d42eebb79|1722902400000|1730682000000|1754006400000|app/components/courses/new.hbs