Virtual tutor merge lecture and exercise

[magnemo]

Description

This pull request contains these completed PBI's:

[Web Create] As a content creator, I want to be able to put an exercise on the section I'm creating [Web Create] As a content creator, I want to be able to put a lecture on the section I'm creating [Web Edit] As a content creator, I want to be able to edit or delete sections, lectures or exercises of the courses I haven't published yet

Changes

All changes have been made in the section edit page (excluding bug fixes).

General changes Translated to Portuguese. A limit of having max 10 exercises + lectures has been implemented.

• Exercises

  • You can now create exercises.
  • You can now see a list of existing exercises.
  • You can now update details of an existing exercise.
  • You can now delete exercises.
  • Exercise answers can be added or removed.
  • Exercise can now have 2-4 answers, where only 1 can be correct.

• Lectures

  • You can now create lectures.
  • You can now see a list of existing lectures.
  • You can now update details of an existing lecture.
  • You can now delete lectures.
  • Lecture creating has a visual drop-zone for file upload. (functionality not implemented yet)
  • A limit of having max 7 lectures has been implemented.

Bug fixes: Courses (Create/Update) now sends a number value for difficulty. Course (Create/Update) now sends a translated response of the categories to the back end.

Checklist

• [x] Code has been tested locally and passes all relevant tests.
• [x] Documentation has been updated to reflect the changes, if applicable.
• [x] Code follows the established coding style and guidelines of the project.
• [x] All new and existing tests related to the changes have passed.
• [x] Any necessary dependencies or new packages have been properly documented.
• [x] Pull request title and description are clear and descriptive.
• [x] Reviewers have been assigned to the pull request.
• [x] Any potential security implications have been considered and addressed.
• [x] Performance impact of the changes has been evaluated, if relevant.

Notes for Reviewers

The backend used for this pull requist is "VirtualTurtor-Merge". This code needs access to mongoDB to function.

Linting gives a lot of errors, we fixed a lot of the existing errors linting gave, but the rest are non trivial errors that don't inhibit the code and are therefore not a priority right no

[SlovenlyTF]

A test on dev is not working. It gets the error: "SyntaxError: Cannot use 'import.meta' outside a module"... Not our test, and the code is working just fine outside the test file. It's just because the test file isn't considered by Node to be a module. As it isn't our test, we don't know how it is supposed to run and how then to modify it to work on short notice.

[MortenMunk]

A test on dev is not working. It gets the error: "SyntaxError: Cannot use 'import.meta' outside a module"... Not our test, and the code is working just fine outside the test file. It's just because the test file isn't considered by Node to be a module. As it isn't our test, we don't know how it is supposed to run and how then to modify it to work on short notice.

The tests should have passed when they were pushed to dev originally, hence the tests not passing now does not make sense, unless conflict has been introduced. I think we should attempt to fix the tests rather than just ignore them, as this defeats the purpose of tests.... Otherwise we should discuss this with the integration team before merging imo.

[JackSMunn]

Approved, and after a small hotfix session, we got it to work.