Andal.LND/frontend-app-learning
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge PR #228 log/unexpected-course-block

Browse Source 
* Commits:
  Log "Section ... has child block" as info, not error
  Log "Unexpected course block type" as info, not error
This commit is contained in:
stvn
2020-09-30 14:59:53 -07:00
parent 42715d3de2 753925ba99
commit e6e5258e5b
2 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/course-home/data/api.js
View File

@@ -1,6 +1,6 @@
import { camelCaseObject, getConfig } from '@edx/frontend-platform';
import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
import { logError } from '@edx/frontend-platform/logging';
import { logInfo } from '@edx/frontend-platform/logging';
function normalizeCourseHomeCourseMetadata(metadata) {
const data = camelCaseObject(metadata);
@@ -53,7 +53,7 @@ export function normalizeOutlineBlocks(courseId, blocks) {
break;
default:
logError(`Unexpected course block type: ${block.type} with ID ${block.id}. Expected block types are course, chapter, and sequential.`);
logInfo(`Unexpected course block type: ${block.type} with ID ${block.id}. Expected block types are course, chapter, and sequential.`);
}
});
@@ -74,7 +74,7 @@ export function normalizeOutlineBlocks(courseId, blocks) {
if (sequenceId in models.sequences) {
models.sequences[sequenceId].sectionId = section.id;
} else {
logError(`Section ${section.id} has child block ${sequenceId}, but that block is not in the list of sequences.`);
logInfo(`Section ${section.id} has child block ${sequenceId}, but that block is not in the list of sequences.`);
}
});
}

8
src/courseware/data/api.js
View File

@@ -1,6 +1,6 @@
import { getConfig, camelCaseObject } from '@edx/frontend-platform';
import { getAuthenticatedHttpClient, getAuthenticatedUser } from '@edx/frontend-platform/auth';
import { logError } from '@edx/frontend-platform/logging';
import { logInfo } from '@edx/frontend-platform/logging';
export function normalizeBlocks(courseId, blocks) {
const models = {
@@ -43,7 +43,7 @@ export function normalizeBlocks(courseId, blocks) {
};
break;
default:
logError(`Unexpected course block type: ${block.type} with ID ${block.id}. Expected block types are course, chapter, sequential, and vertical.`);
logInfo(`Unexpected course block type: ${block.type} with ID ${block.id}. Expected block types are course, chapter, sequential, and vertical.`);
}
});
@@ -64,7 +64,7 @@ export function normalizeBlocks(courseId, blocks) {
if (sequenceId in models.sequences) {
models.sequences[sequenceId].sectionId = section.id;
} else {
logError(`Section ${section.id} has child block ${sequenceId}, but that block is not in the list of sequences.`);
logInfo(`Section ${section.id} has child block ${sequenceId}, but that block is not in the list of sequences.`);
}
});
}
@@ -76,7 +76,7 @@ export function normalizeBlocks(courseId, blocks) {
if (unitId in models.units) {
models.units[unitId].sequenceId = sequence.id;
} else {
logError(`Sequence ${sequence.id} has child block ${unitId}, but that block is not in the list of units.`);
logInfo(`Sequence ${sequence.id} has child block ${unitId}, but that block is not in the list of units.`);
}
});
}