Skip to content
Snippets Groups Projects

Hotfixes2

All threads resolved!
Merged
Otto Visserrequested to merge
hotfixes2 into development
All threads resolved!

What does this mr do?

Fix adding users to a course (e.g.: add teacher to course)

Actions taken to fix bug

Fix autowired mess of user/userservice that was used for checking for valid netIDs

Does this MR meet the acceptance criteria?

  • I have added a changelog entry to reflect the significant changes I made and the bug I fixed.
  • A test was created to test the bug.
  • I have updated the documentation accordingly.
  • I adhere to the style guide.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Sorry for missing all the thymeleaf changes during my refactorings... Sadly IntelliJ can't reliably and statically provide errors for those :(

  • Chris Lemaire resolved all threads

    resolved all threads

  • Chris Lemaire approved this merge request

    approved this merge request

  • Chris Lemaire started a merge train

    started a merge train

  • merged

  • Chris Lemaire mentioned in commit e540d91c

    mentioned in commit e540d91c

  • Please register or sign in to reply
    Loading