Skip to content
This repository was archived by the owner on Apr 16, 2023. It is now read-only.

Issue #13 - Course subscription#32

Open
mirceadino wants to merge 28 commits intobmariesan:masterfrom
al3ks1002:course-subscription
Open

Issue #13 - Course subscription#32
mirceadino wants to merge 28 commits intobmariesan:masterfrom
al3ks1002:course-subscription

Conversation

@mirceadino
Copy link

@mirceadino mirceadino commented Jan 27, 2018

Description

  • Students should have a list of available courses
  • Students can subscribe to available courses
  • Students can see the history of courses they’ve attended
  • Students can see their course progress based on assignments, exams, received grades
  • Students should be able to subscribe to courses (from none to unlimited)
  • Teachers should set a limit of students per course, once the limit is reached course subscriptions are disabled

Checklist

  • Code compiles correctly
  • Created tests for the change (if possible)
  • All tests passing
  • Any merge conflicts should be solved before merging the pull request

al3ks1002 and others added 28 commits December 6, 2017 14:59
Added exceptions for the following use cases
  - a student tries to register to a course to which he/she is already
    subscribed to.
  - a student tries to register to a course which has no slots left
Added souts for debuging purposes when entities are transformed into
dtos.
- Moved DTO related utility functions in static functions in StudentDto
  and CourseDto.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants