Skip to content

Commit 7c49c9e

Browse files
authored
Merge pull request #266 from TechnologyEnhancedLearning/RC
Rc
2 parents 8646677 + 4aea339 commit 7c49c9e

File tree

6 files changed

+10
-23
lines changed

6 files changed

+10
-23
lines changed

LearningHub.Nhs.WebUI/Controllers/Api/ScormController.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ private async Task<bool> Commit(SCO scoObject)
260260

261261
// Persist update.
262262
await this.activityService.UpdateScormActivityAsync(scoObject);
263-
if (scoObject.LessonStatusId == 3)
263+
if (scoObject.LessonStatusId == ScormLessionStatus.ActivityStatusId(ScormLessionStatus.Completed) || scoObject.LessonStatusId == ScormLessionStatus.ActivityStatusId(ScormLessionStatus.Passed))
264264
{
265265
await this.activityService.CompleteScormActivity(scoObject);
266266
}

LearningHub.Nhs.WebUI/Models/MyLearningViewModel.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,8 @@ public List<CheckboxListItemViewModel> StatusFilterCheckbox()
184184
{
185185
var checkboxes = new List<CheckboxListItemViewModel>()
186186
{
187-
new CheckboxListItemViewModel("Complete", "Complete", null),
188-
new CheckboxListItemViewModel("Incomplete", "InProgress", null),
187+
new CheckboxListItemViewModel("Complete", "Completed", null),
188+
new CheckboxListItemViewModel("Incomplete", "In Progress", null),
189189
new CheckboxListItemViewModel("Passed", "Passed", null),
190190
new CheckboxListItemViewModel("Failed", "Failed", null),
191191
new CheckboxListItemViewModel("Downloaded", "Downloaded", null),

LearningHub.Nhs.WebUI/Views/MyLearning/_filterSummary.cshtml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,11 +53,11 @@
5353
}
5454
@if (Model.Complete)
5555
{
56-
filtersummary = filtersummary + $" <strong class='nhsuk-tag'>Complete</strong>";
56+
filtersummary = filtersummary + $" <strong class='nhsuk-tag'>Completed</strong>";
5757
}
5858
@if (Model.Incomplete)
5959
{
60-
filtersummary = filtersummary + $" <strong class='nhsuk-tag'>InProgress</strong>";
60+
filtersummary = filtersummary + $" <strong class='nhsuk-tag'>In Progress</strong>";
6161
}
6262
@if (Model.Passed)
6363
{

LearningHub.Nhs.WebUI/Views/Resource/_ResourceItem.cshtml

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -161,23 +161,7 @@
161161
</div>
162162
}
163163
}
164-
@* Html Resource *@
165-
@if (canShowMoreDetails && resourceItem.ResourceTypeEnum == ResourceTypeEnum.Html)
166-
{
167-
<div id="resourcePg" class="nhsuk-card nhsuk-bg-light-blue">
168-
<div class="nhsuk-card__content">
169-
<a v-on:click=checkUserCertificateAvailability(@resourceItem.Id) href="@($"{Context.Request.Scheme}://{Context.Request.Host}/resource/html/{resourceItem.Id}/index.html")"
170-
onclick="@($"window.open(this.href, 'scormDebugWindow', 'status=no,location=no,toolbar=no,menubar=no,dependent=no,width={resourceItem.HtmlDetails.PopupWidth},height={resourceItem.HtmlDetails.PopupHeight}'); return false;")">
171-
<button class="nhsuk-button">
172-
Launch HTML resource
173-
</button>
174-
</a>
175-
<p>This resource will launch in a new window</p>
176-
</div>
177-
</div>
178-
}
179-
180-
164+
181165
@if (canShowMoreDetails && resourceItem.ResourceTypeEnum == ResourceTypeEnum.Video || resourceItem.ResourceTypeEnum == ResourceTypeEnum.Audio || resourceItem.ResourceTypeEnum == ResourceTypeEnum.Scorm ||
182166
resourceItem.ResourceTypeEnum == ResourceTypeEnum.Case || resourceItem.ResourceTypeEnum == ResourceTypeEnum.Assessment)
183167
{

WebAPI/LearningHub.Nhs.Database/Stored Procedures/Activity/GetUserLearningActivities.sql

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,8 @@ FROM (
245245
)
246246
)
247247
)
248-
248+
OR
249+
([Res].[ResourceTypeId] IN (6) AND [ResourceActivity].[ActivityStatusId] = 3)
249250
--OR
250251
--(
251252
-- ([Res].[ResourceTypeId] IN (1,5,10,12) AND [ResourceActivity].[ActivityStatusId] = 3)

WebAPI/LearningHub.Nhs.Database/Stored Procedures/Activity/GetUserLearningActivitiesCount.sql

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -160,6 +160,8 @@ FROM (
160160
)
161161
)
162162
)
163+
OR
164+
([Res].[ResourceTypeId] IN (6) AND [ResourceActivity].[ActivityStatusId] = 3)
163165

164166
--OR
165167
--(

0 commit comments

Comments
 (0)