Skip to content

Commit 1c29226

Browse files
committed
Rename and move isMinigame to assessment config in admin panel
1 parent 7290bc8 commit 1c29226

File tree

8 files changed

+35
-2
lines changed

8 files changed

+35
-2
lines changed

src/commons/application/actions/__tests__/SessionActions.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,7 @@ test('setAssessmentConfigurations generates correct action object', () => {
264264
isManuallyGraded: true,
265265
isGradingAutoPublished: false,
266266
displayInDashboard: true,
267+
isMinigame: false,
267268
hasTokenCounter: false,
268269
hasVotingFeatures: false,
269270
hoursBeforeEarlyXpDecay: 48,
@@ -275,6 +276,7 @@ test('setAssessmentConfigurations generates correct action object', () => {
275276
isManuallyGraded: true,
276277
isGradingAutoPublished: false,
277278
displayInDashboard: true,
279+
isMinigame: false,
278280
hasTokenCounter: false,
279281
hasVotingFeatures: false,
280282
hoursBeforeEarlyXpDecay: 48,
@@ -286,6 +288,7 @@ test('setAssessmentConfigurations generates correct action object', () => {
286288
isManuallyGraded: true,
287289
isGradingAutoPublished: false,
288290
displayInDashboard: true,
291+
isMinigame: false,
289292
hasTokenCounter: false,
290293
hasVotingFeatures: false,
291294
hoursBeforeEarlyXpDecay: 48,

src/commons/assessment/AssessmentTypes.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ export type Assessment = {
9292
globalDeployment?: Library; // For mission control
9393
graderDeployment?: Library; // For mission control
9494
hasTokenCounter?: boolean;
95-
onFinish?: string;
95+
isMinigame?: boolean;
9696
id: number;
9797
longSummary: string;
9898
missionPDF: string;
@@ -106,6 +106,7 @@ export type AssessmentConfiguration = {
106106
isManuallyGraded: boolean;
107107
isGradingAutoPublished: boolean;
108108
displayInDashboard: boolean;
109+
isMinigame: boolean;
109110
hoursBeforeEarlyXpDecay: number;
110111
earlySubmissionXp: number;
111112
hasTokenCounter: boolean;

src/commons/assessmentWorkspace/AssessmentWorkspace.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -602,6 +602,8 @@ const AssessmentWorkspace: React.FC<AssessmentWorkspaceProps> = props => {
602602
};
603603
};
604604

605+
console.log(assessment);
606+
605607
/**
606608
* controlBarProps() will only be called when assessment is not undefined
607609
* (see 'Rendering Logic' below), thus it is okay to use assessment!
@@ -700,7 +702,7 @@ const AssessmentWorkspace: React.FC<AssessmentWorkspaceProps> = props => {
700702
}
701703
questionProgress={questionProgress}
702704
key="next_question"
703-
submitOnFinish={assessment.onFinish === 'submitAndReturnToGame'}
705+
submitOnFinish={assessment.isMinigame}
704706
/>
705707
);
706708

src/commons/assessmentWorkspace/__tests__/AssessmentWorkspace.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ const defaultProps = assertType<AssessmentWorkspaceProps>()({
2121
isManuallyGraded: true,
2222
isGradingAutoPublished: false,
2323
displayInDashboard: true,
24+
isMinigame: false,
2425
hasTokenCounter: false,
2526
hasVotingFeatures: false,
2627
hoursBeforeEarlyXpDecay: 48,

src/commons/mocks/AssessmentMocks.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
2121
isManuallyGraded: true,
2222
isGradingAutoPublished: false,
2323
displayInDashboard: true,
24+
isMinigame: false,
2425
hoursBeforeEarlyXpDecay: 48,
2526
hasTokenCounter: false,
2627
hasVotingFeatures: false,
@@ -32,6 +33,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
3233
isManuallyGraded: true,
3334
isGradingAutoPublished: false,
3435
displayInDashboard: true,
36+
isMinigame: false,
3537
hoursBeforeEarlyXpDecay: 48,
3638
hasTokenCounter: false,
3739
hasVotingFeatures: false,
@@ -43,6 +45,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
4345
isManuallyGraded: false,
4446
isGradingAutoPublished: true,
4547
displayInDashboard: true,
48+
isMinigame: false,
4649
hoursBeforeEarlyXpDecay: 48,
4750
hasTokenCounter: false,
4851
hasVotingFeatures: false,
@@ -54,6 +57,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
5457
isManuallyGraded: true,
5558
isGradingAutoPublished: false,
5659
displayInDashboard: true,
60+
isMinigame: false,
5761
hoursBeforeEarlyXpDecay: 48,
5862
hasTokenCounter: false,
5963
hasVotingFeatures: true,
@@ -65,6 +69,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
6569
isManuallyGraded: true,
6670
isGradingAutoPublished: false,
6771
displayInDashboard: true,
72+
isMinigame: false,
6873
hoursBeforeEarlyXpDecay: 48,
6974
hasTokenCounter: false,
7075
hasVotingFeatures: false,
@@ -78,6 +83,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
7883
isManuallyGraded: true,
7984
isGradingAutoPublished: false,
8085
displayInDashboard: true,
86+
isMinigame: false,
8187
hoursBeforeEarlyXpDecay: 48,
8288
hasTokenCounter: false,
8389
hasVotingFeatures: false,
@@ -89,6 +95,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
8995
isManuallyGraded: true,
9096
isGradingAutoPublished: false,
9197
displayInDashboard: true,
98+
isMinigame: false,
9299
hoursBeforeEarlyXpDecay: 48,
93100
hasTokenCounter: false,
94101
hasVotingFeatures: false,
@@ -100,6 +107,7 @@ export const mockAssessmentConfigurations: AssessmentConfiguration[][] = [
100107
isManuallyGraded: true,
101108
isGradingAutoPublished: false,
102109
displayInDashboard: true,
110+
isMinigame: false,
103111
hoursBeforeEarlyXpDecay: 48,
104112
hasTokenCounter: false,
105113
hasVotingFeatures: false,

src/commons/sagas/BackendSaga.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -957,6 +957,7 @@ const newBackendSagaTwo = combineSagaHandlers(sagaActions, {
957957
isManuallyGraded: true,
958958
isGradingAutoPublished: false,
959959
displayInDashboard: true,
960+
isMinigame: false,
960961
hoursBeforeEarlyXpDecay: 0,
961962
hasTokenCounter: false,
962963
hasVotingFeatures: false,

src/commons/sagas/__tests__/BackendSaga.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ const mockAssessmentConfigurations: AssessmentConfiguration[] = [
177177
isManuallyGraded: true,
178178
isGradingAutoPublished: false,
179179
displayInDashboard: true,
180+
isMinigame: false,
180181
hoursBeforeEarlyXpDecay: 48,
181182
hasTokenCounter: false,
182183
hasVotingFeatures: false,
@@ -188,6 +189,7 @@ const mockAssessmentConfigurations: AssessmentConfiguration[] = [
188189
isManuallyGraded: true,
189190
isGradingAutoPublished: false,
190191
displayInDashboard: true,
192+
isMinigame: false,
191193
hoursBeforeEarlyXpDecay: 48,
192194
hasTokenCounter: false,
193195
hasVotingFeatures: false,
@@ -199,6 +201,7 @@ const mockAssessmentConfigurations: AssessmentConfiguration[] = [
199201
isManuallyGraded: false,
200202
isGradingAutoPublished: true,
201203
displayInDashboard: false,
204+
isMinigame: false,
202205
hoursBeforeEarlyXpDecay: 48,
203206
hasTokenCounter: false,
204207
hasVotingFeatures: false,
@@ -210,6 +213,7 @@ const mockAssessmentConfigurations: AssessmentConfiguration[] = [
210213
isManuallyGraded: false,
211214
isGradingAutoPublished: false,
212215
displayInDashboard: false,
216+
isMinigame: false,
213217
hoursBeforeEarlyXpDecay: 48,
214218
hasTokenCounter: false,
215219
hasVotingFeatures: true,
@@ -221,6 +225,7 @@ const mockAssessmentConfigurations: AssessmentConfiguration[] = [
221225
isManuallyGraded: true,
222226
isGradingAutoPublished: false,
223227
displayInDashboard: false,
228+
isMinigame: false,
224229
hoursBeforeEarlyXpDecay: 48,
225230
hasTokenCounter: false,
226231
hasVotingFeatures: false,
@@ -1042,6 +1047,7 @@ describe('Test CREATE_COURSE action', () => {
10421047
isManuallyGraded: true,
10431048
isGradingAutoPublished: false,
10441049
displayInDashboard: true,
1050+
isMinigame: false,
10451051
hoursBeforeEarlyXpDecay: 0,
10461052
hasTokenCounter: false,
10471053
hasVotingFeatures: false,

src/pages/academy/adminPanel/subcomponents/assessmentConfigPanel/AssessmentConfigPanel.tsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ const defaultAssessmentConfig: AssessmentConfiguration = {
3131
isManuallyGraded: true,
3232
isGradingAutoPublished: false,
3333
displayInDashboard: true,
34+
isMinigame: false,
3435
hoursBeforeEarlyXpDecay: 0,
3536
hasTokenCounter: false,
3637
hasVotingFeatures: false,
@@ -131,6 +132,7 @@ const AssessmentConfigPanel: WithImperativeApi<
131132

132133
/* eslint-disable react-hooks/exhaustive-deps */
133134
const setDisplayInDashboard = useCallback(valueSetter('displayInDashboard'), []);
135+
const setIsMinigame = useCallback(valueSetter('isMinigame'), []);
134136
const setHasTokenCounter = useCallback(valueSetter('hasTokenCounter'), []);
135137
const setHasVotingFeatures = useCallback(valueSetter('hasVotingFeatures'), []);
136138
const setEarlyXp = useCallback(valueSetter('earlySubmissionXp'), []);
@@ -201,6 +203,15 @@ const AssessmentConfigPanel: WithImperativeApi<
201203
field: 'displayInDashboard'
202204
}
203205
},
206+
{
207+
headerName: 'Is Minigame',
208+
field: 'isMinigame',
209+
cellRenderer: BooleanCell,
210+
cellRendererParams: {
211+
setStateHandler: setIsMinigame,
212+
field: 'isMinigame'
213+
}
214+
},
204215
{
205216
headerName: 'Voting Features*',
206217
field: 'hasVotingFeatures',

0 commit comments

Comments
 (0)