robinwilliam.hundt authored 7 years ago and Jan Maximilian Michal committed 7 years ago

 * Defnied more APIendpoints for feedback and subscriptions
 * Added the feedback spec to the docs folder
 * Removed tutor and reviewer model. Closes #43 and #68

* The mechanism proposed should work as follows:
  * Tutors can subscribe to certain submission categries (currently
    this includes exam, student or type specific submissions).
    If the set of submissions to corrent is small (student) all
    submissions of that category are reserved for that tutor.
  * A reviewer should also be able to subscribe other users (delegation)
  * A subscription contains assignments or creates them:
    * Only one assignment per user may be active.
    * No new assignments can be added to a subscription after it was
      created while another assignment is present for that subscription.
    * An assignment delegates a submission to a tutor.
    * An active assignment indicates that the tutor is working on that
      assignment
    * After an assignment was finished it is deleted (or archived).
* Upgraded to Django 2.0
* Closes #66, #53.

* The mechanism remains partially incomplete as the progress in
  !67-create-new-model-feedbackline-and-integrate-it is blocking
  progress. Several tests for the API endpoint need to be written
  including. More validation and constraints might have to be added.

Restructured front end code into components and pages
Components should be as dumb and generic as possible. Pages should dispatch actions, pass props to components etc,
Student page now gets submission and submissiontyp from api and displays those to the student
Added information which submissions have been viewed

This endpoint returns the time delta of the JWT Tokens as specified in the settings file #22

This endpoint will return the role of the authenticated user, 'Student', 'Tutor' or 'Reviewer'

The extra functionality offered by the ViewSet is not needed in this place and caused unexpected behavior (it returned an array containing an object, breaking the StudentView). For this very simple View imo explicit is better than implicit.

* using gevnts for async requests handling of gunicorn
* serving the static frontend via gunicorn
* split requirements into dev and general
{

* IT waS A FUCKIng tyPO
* ran isort and flake8 and fixed reported issues

* no url configuration needed simply register viewset with router
* now using DefaultRouter, meaning api root is now browsable
* merged some views
* makes it easier to include api schema later
* Ran isort and updated docstring

- changed unittester from ./manage.py test to pytest in CI
- implemented tests for various views see previous commits
- coverage should now produce html output for test coverage

* ran isort on all test files

Backend
- Added isStudent permission
- Added serializers apropriate for StudentPage
- Added api-endpoint urls
- Added views regarding StudentPage
- Migrated the Django Session Authentication to djangorestframework-jwt authentication

Frontend
- Migrated from vue-resource to axios because of better documentation & features
- Added Login Component
- Added First Part of StudentPage containing overview of exam and subissions

Closes #41
References #39

- ran isort to properly sort imports
- reformatter some classes

Deleted templates, fixtures, static files, urls, views

Backend
- Added isStudent permission
- Added serializers apropriate for StudentPage
- Added api-endpoint urls
- Added views regarding StudentPage
- Migrated the Django Session Authentication to djangorestframework-jwt authentication

Frontend
- Migrated from vue-resource to axios because of better documentation & features
- Added Login Component
- Added First Part of StudentPage containing overview of exam and subissions

Closes #41
References #39

The project has been refactored so far as the current state of master has been placed under backend/ .
Some changes to the .gitignore and Makefile have been made to accomodate this refactor.

and closes #23. Other minor changes.