Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue/k UI 1601 remove fields #396

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
+16 −63
Open

Issue/k UI 1601 remove fields #396

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
606b503
feat(KUI-1601): remove possibilityToAddition
amirhossein-haerian Nov 21, 2024
4740e4b
feat(KUI-1601): update test cases
amirhossein-haerian Nov 21, 2024
dd4f8c7
feat(KUI-1601): remove possibilityToCompletion
amirhossein-haerian Nov 21, 2024
e1d750f
feat(KUI-1601): remove requiredEquipment
amirhossein-haerian Nov 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 3 additions & 6 deletions i18n/messages.en.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -185,8 +185,7 @@ module.exports = {
},
equipment: {
body: `<p><b>Equipment</b> describes what equipment is not provided by KTH and is needed to follow the course offering.</p>
<p><b>This information helps the student</b> acquire the right equipment before the start of the course offering.</p>
<p><b>The content is retrieved from Kopps</b> if there is information about equipment already entered. You can choose to retain that information in your course memo or add new information.</p>`,
<p><b>This information helps the student</b> acquire the right equipment before the start of the course offering.</p>`,
help: `<p>Under this section, you describe the equipment that students need to complete the course, if it is not provided by KTH. Writing tools and literature are not considered equipment.</p>`,
},
examiner: {
Expand Down Expand Up @@ -293,15 +292,13 @@ module.exports = {
},
possibilityToAddition: {
body: `<p><b>Opportunity to raise an approved grade via renewed examination</b> describes if it is possible to raise an approved grade on this course and the terms for raising the grade.</p>
<p><b>This information helps the student</b> to understand if it is possible to raise an approved grade via renewed examination in this course offering.</p>
<p><b>Content is retrieved from Kopps</b> if there is information about Opportunity to raise an approved grade via renewed examination entered. You can choose to retain that information in your course memo or add new information.</p>`,
<p><b>This information helps the student</b> to understand if it is possible to raise an approved grade via renewed examination in this course offering.</p>`,
help: `<p>Under this section, you describe if it is possible to raise an approved grade via renewed examination on this course offering and the terms for that. Also describe how the student can apply for a renewed examination.
If it is not possible to raise an approved grade via renewed examination on this course offering it may be beneficial to indicate this.</p>`,
},
possibilityToCompletion: {
body: `<p><b>Opportunity to complete the requirements via supplementary examination</b> describes whether there is a chance for supplementary examination in this course offering and if completion can be done only up to a grade E or to a higher grade.</p>
<p><b>This information helps the student</b> to understand if it is possible to complete the requirements via supplementary examination and, if so, the conditions for completion in this course offering.</p>
<p><b>Content is retrieved from Kopps</b> if there is information about Opportunity to complete the requirements via supplementary examination entered. You can choose to retain that information in your course memo or add new information.</p>`,
<p><b>This information helps the student</b> to understand if it is possible to complete the requirements via supplementary examination and, if so, the conditions for completion in this course offering.</p>`,
help: `<p>Under this section, you describe if it is possible to complete the requirements via supplementary examination on this course offering and, if so, the applicable conditions. Also describe if it is possible to complete the requirements via supplementary examination on each module.</p>
<p>All modules in a course with differentiated grading scale must offer an opportunity to complete the requirements via supplementary examination from Fx to Pass. Opportunity to complete the requirements via supplementary examination can be offered by the examiner for modules with grading scale P/F.</p>
<p>It is often better to describe opportunity to complete the requirements via supplementary examination for the modules under the heading Examination.</p>`,
Expand Down
9 changes: 3 additions & 6 deletions i18n/messages.se.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -174,8 +174,7 @@ module.exports = {
},
equipment: {
body: `<p><b>Utrustning</b> beskriver vilken utrustning som inte tillhandahålls av KTH och som behövs för att kunna följa kursomgången.</p>
<p><b>Informationen hjälper studenten</b> att skaffa rätt utrustning inför kursomgångens start.</p>
<p><b>Innehåll hämtas från Kopps</b> om det finns uppgifter om utrustning inlagt. Du kan välja att behålla den informationen i ditt kurs-PM, eller skriva in ny information.</p>`,
<p><b>Informationen hjälper studenten</b> att skaffa rätt utrustning inför kursomgångens start.</p>`,
help: `<p>Här beskriver du vilken utrustning som studenten behöver för att genomföra kursen, om den inte tillhandahålls av KTH. Skrivdon och litteratur räknas inte till utrustning.</p>`,
},
examination: {
Expand Down Expand Up @@ -282,14 +281,12 @@ module.exports = {
},
possibilityToAddition: {
body: `<p><b>Möjlighet till plussning</b> beskriver om plussning är möjligt på kursomgången, eventuella villkor för plussning och hur plussning går till.</p>
<p><b>Informationen hjälper studenten</b> att förstå om det finns möjlighet till plussning på denna kursomgång.</p>
<p><b>Innehåll hämtas från Kopps</b> om det finns uppgifter om möjlighet till plussning inlagt. Du kan välja att behålla den informationen i ditt kurs-PM, eller skriva in ny information.</p>`,
<p><b>Informationen hjälper studenten</b> att förstå om det finns möjlighet till plussning på denna kursomgång.</p>`,
help: `<p>Här beskriver du om det finns möjlighet till plussning på denna kursomgång och i så fall vilka villkor som gäller för det och hur studenten ansöker om att få plussa. Om möjlighet till plussning inte är möjlig för denna kursomgång kan det vara bra att ange detta.</p>`,
},
possibilityToCompletion: {
body: `<p><b>Möjlighet till komplettering</b> beskriver om det finns möjlighet till komplettering på denna kursomgång och om komplettering kan göras endast till betyg E eller till högre betyg.</p>
<p><b>Informationen hjälper studenten</b> att förstå om det finns möjlighet till komplettering på denna kursomgång och i så fall villkoren för komplettering.</p>
<p><b>Innehåll hämtas från Kopps</b> om det finns uppgifter om möjlighet till komplettering inlagt. Du kan välja att behålla den informationen i ditt kurs-PM, eller skriva in ny information.</p>`,
<p><b>Informationen hjälper studenten</b> att förstå om det finns möjlighet till komplettering på denna kursomgång och i så fall villkoren för komplettering.</p>`,
help: `<p>Under rubriken beskriver du om det finns möjlighet till komplettering på denna kursomgång och i så fall vilka villkor som gäller. Beskriv också vad som gäller för kursens moduler.</p>
<p>För modul med betygsskala A-F måste möjlighet till komplettering från Fx till Godkänt ges. För modul med betygsskala P/F kan det erbjudas av examinator.</p>
<p>Det kan i många fall vara mer tydligt att istället beskriva möjlighet till komplettering om varje modul under rubriken Examination.</p>`,
Expand Down
9 changes: 3 additions & 6 deletions server/controllers/memoCtrl.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ const combineDefaultValues = (freshMemoData, koppsFreshData) => {
const updatedWithDefaults = {
...freshMemoData,
// eslint-disable-next-line no-use-before-define
equipment: equipment || koppsFreshData.equipmentTemplate || '',
equipment: equipment || '',
scheduleDetails: scheduleDetails || '',
literature: literature || koppsFreshData.literatureTemplate || '',
possibilityToCompletion: possibilityToCompletion || koppsFreshData.possibilityToCompletionTemplate || '',
possibilityToAddition: possibilityToAddition || koppsFreshData.possibilityToAdditionTemplate || '',
possibilityToCompletion: possibilityToCompletion || '',
possibilityToAddition: possibilityToAddition || '',
}

return updatedWithDefaults
Expand All @@ -33,10 +33,7 @@ const combineDefaultValues = (freshMemoData, koppsFreshData) => {
const removeTemplatesFromKoppsFreshData = async koppsFreshData => {
// no map()
// to send cleaned up koppsFreshData to client side end then to api
await delete koppsFreshData.equipmentTemplate
await delete koppsFreshData.literatureTemplate
await delete koppsFreshData.possibilityToCompletionTemplate
await delete koppsFreshData.possibilityToAdditionTemplate
return koppsFreshData
}

Expand Down
3 changes: 0 additions & 3 deletions server/koppsApi.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -222,11 +222,8 @@ function _getCommonInfo(resBody) {
gradingScale,
title: c.title || '',
titleOther: c.titleOther || '',
possibilityToCompletionTemplate: c.possibilityToCompletion || '',
possibilityToAdditionTemplate: c.possibilityToAddition || '',
schemaUrls: schemaUrls || '',
literatureTemplate: c.courseLiterature || '',
equipmentTemplate: c.requiredEquipment || '',
}
}

Expand Down
9 changes: 0 additions & 9 deletions test/unit/kopps-parseSyllabus.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,6 @@ test('parse an older syllabus 20192 information for 20212 without breaking if TE
"credits": 7.5,
"departmentName": "ITM/Hållbar produktionsutveckling",
"educationalTypeId": null,
"equipmentTemplate": "",
"ethicalApproach": "<ul><li>Vid grupparbete har alla i gruppen ansvar för gruppens arbete.</li><li>Vid examination ska varje student ärligt redovisa hjälp som erhållits och källor som använts.</li><li>Vid muntlig examination ska varje student kunna redogöra för hela uppgiften och hela lösningen.</li></ul>",
"examComments": "Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p>",
"examination": "<p><ul><li>INL1 - Inlämningsuppgifter, 3,5 hp, Betygsskala: A, B, C, D, E, FX, F</li><li>TEN1 - Skriftlig tentamen, 4,0 hp, Betygsskala: A, B, C, D, E, FX, F</li></ul></p><p>Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p></p>",
Expand All @@ -225,8 +224,6 @@ test('parse an older syllabus 20192 information for 20212 without breaking if TE
"otherRequirementsForFinalGrade": "<p>N&#228;rvaro vid specificerade obligatoriska kurstillf&#228;llen (eller genomf&#246;rd kompensationsuppgift till dessa) &#228;r krav f&#246;r slutlig betygs&#228;ttning.</p>",
"permanentDisability": "<p>Om du har en funktionsnedsättning kan du få stöd via Funka:</p>
<p><a href="https://www.kth.se/student/stod/studier/funktionsnedsattning/funka">Funka- stöd för studenter med funktionsnedsättningar</a></p>",
"possibilityToAdditionTemplate": "",
"possibilityToCompletionTemplate": "",
"recruitmentText": "",
"schemaUrls": [
"https://www.kth.se/social/course/ML1616/subgroup/ht-2022-tiips-7/calendar/",
Expand Down Expand Up @@ -258,7 +255,6 @@ test('parse the latest syllabus 20222 for 20222', () => {
"credits": 7.5,
"departmentName": "ITM/Hållbar produktionsutveckling",
"educationalTypeId": null,
"equipmentTemplate": "",
"ethicalApproach": "<ul><li>Vid grupparbete har alla i gruppen ansvar för gruppens arbete.</li><li>Vid examination ska varje student ärligt redovisa hjälp som erhållits och källor som använts.</li><li>Vid muntlig examination ska varje student kunna redogöra för hela uppgiften och hela lösningen.</li></ul>",
"examComments": "Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p><p>TENA &#228;r ett valfritt examinationsmoment f&#246;r att uppn&#229; h&#246;gre betyg &#228;n E p&#229; kursen.</p>",
"examination": "<p><ul><li>INL1 - Inlämningsuppgifter, 3,5 hp, Betygsskala: A, B, C, D, E, FX, F</li><li>KON1 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON2 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON3 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON4 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>TENA - Hemtentamen, Betygsskala: A, B, C, D, E, FX, F</li></ul></p><p>Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p><p>TENA &#228;r ett valfritt examinationsmoment f&#246;r att uppn&#229; h&#246;gre betyg &#228;n E p&#229; kursen.</p></p>",
Expand All @@ -269,8 +265,6 @@ test('parse the latest syllabus 20222 for 20222', () => {
"otherRequirementsForFinalGrade": "<p>N&#228;rvaro vid specificerade obligatoriska kurstillf&#228;llen (eller genomf&#246;rd kompensationsuppgift till dessa) &#228;r krav f&#246;r slutlig betygs&#228;ttning.</p>",
"permanentDisability": "<p>Om du har en funktionsnedsättning kan du få stöd via Funka:</p>
<p><a href="https://www.kth.se/student/stod/studier/funktionsnedsattning/funka">Funka- stöd för studenter med funktionsnedsättningar</a></p>",
"possibilityToAdditionTemplate": "",
"possibilityToCompletionTemplate": "",
"recruitmentText": "",
"schemaUrls": [
"https://www.kth.se/social/course/ML1616/subgroup/ht-2022-tiips-7/calendar/",
Expand Down Expand Up @@ -298,7 +292,6 @@ test('parse the latest syllabus 20222 for 20231', () => {
"credits": 7.5,
"departmentName": "ITM/Hållbar produktionsutveckling",
"educationalTypeId": null,
"equipmentTemplate": "",
"ethicalApproach": "<ul><li>Vid grupparbete har alla i gruppen ansvar för gruppens arbete.</li><li>Vid examination ska varje student ärligt redovisa hjälp som erhållits och källor som använts.</li><li>Vid muntlig examination ska varje student kunna redogöra för hela uppgiften och hela lösningen.</li></ul>",
"examComments": "Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p><p>TENA &#228;r ett valfritt examinationsmoment f&#246;r att uppn&#229; h&#246;gre betyg &#228;n E p&#229; kursen.</p>",
"examination": "<p><ul><li>INL1 - Inlämningsuppgifter, 3,5 hp, Betygsskala: A, B, C, D, E, FX, F</li><li>KON1 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON2 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON3 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>KON4 - Kontrollskrivning, 1,0 hp, Betygsskala: P, F</li><li>TENA - Hemtentamen, Betygsskala: A, B, C, D, E, FX, F</li></ul></p><p>Examinator beslutar, baserat på rekommendation från KTH:s samordnare för funktionsnedsättning, om eventuell anpassad examination för studenter med dokumenterad, varaktig funktionsnedsättning. <br><br>Examinator får medge annan examinationsform vid omexamination av enstaka studenter.<p>Examinator beslutar, i samr&#229;d med KTH:s samordnare f&#246;r funktionsneds&#228;ttning (Funka), om eventuell anpassad examination f&#246;r studenter med dokumenterad, varaktig funktionsneds&#228;ttning. Examinator f&#229;r medge annan examinationsform vid omexamination av enstaka studenter.</p><p>TENA &#228;r ett valfritt examinationsmoment f&#246;r att uppn&#229; h&#246;gre betyg &#228;n E p&#229; kursen.</p></p>",
Expand All @@ -309,8 +302,6 @@ test('parse the latest syllabus 20222 for 20231', () => {
"otherRequirementsForFinalGrade": "<p>N&#228;rvaro vid specificerade obligatoriska kurstillf&#228;llen (eller genomf&#246;rd kompensationsuppgift till dessa) &#228;r krav f&#246;r slutlig betygs&#228;ttning.</p>",
"permanentDisability": "<p>Om du har en funktionsnedsättning kan du få stöd via Funka:</p>
<p><a href="https://www.kth.se/student/stod/studier/funktionsnedsattning/funka">Funka- stöd för studenter med funktionsnedsättningar</a></p>",
"possibilityToAdditionTemplate": "",
"possibilityToCompletionTemplate": "",
"recruitmentText": "",
"schemaUrls": [
"https://www.kth.se/social/course/ML1616/subgroup/ht-2022-tiips-7/calendar/",
Expand Down
Loading