Meeting 15 may copy

[SamuelVch98]

• Updating the code to take account of the comments made by the first PR
• data.py : Add a 'data.py' file containing important data not stored in the DB
• templates/user_profile.html : merge tables to add preferences in a single dynamic table
  • See only the courses in your organisation in the preferences
• templates/course_info.html + db.py : delete the 'load_needed' field in the database and in the view
• templates/courses.html : display the teachers for each course
• templates/layout.html : creation of drop-down menus for users and the course + creation of a confirmation modal for the change of year

[anthonygego]

Those minor comments should be fixed quickly. Please fix them as soon as possible so that we can merge and be able to read the next PRs diffs correctly. Maybe you will need to rebase them once this one is merged.

[SamuelVch98]

These are the last comments after code testing. There are no length specified for the strings in the models: what DBMS are you using ? Ideally, we would like to rely on MySQL/MariaDB .

I use PostgreSQL but it's true that I should add a size limit for strings. Should I make the change straight away or can I add it in another PR?

[anthonygego]

LGTM. Please rebase the next PRs to master. Thanks