Skip to content
Snippets Groups Projects

Resolve "Overview: Replace the course link with a button on the overview"

What does this mr do?

Adds a view course button instead of clicking on the course name, this way enrolling and viewing a course look the same and it is more intuitive. Also cleaned up the thyme leaf template by removing redundant code.

Screenshots

after

Screenshot_2020-05-07_at_11.42.35

Does this MR meet the acceptance criteria?

  • I have added a changelog entry to reflect the significant changes I made.
  • Tests were created to test the feature.
  • I have updated the documentation accordingly.
  • I adhere to the style guide.

Closes #121 (closed)

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
  • Thijs Nulle added 1 commit

    added 1 commit

    • 6c9e44e7 - Refactor conditions in course list thymeleaf template

    Compare with previous version

  • Otto Visser resolved all threads

    resolved all threads

  • Otto Visser approved this merge request

    approved this merge request

  • Otto Visser started a merge train

    started a merge train

  • Otto Visser mentioned in commit 0dadbdfb

    mentioned in commit 0dadbdfb

  • merged

  • unassigned @tnulle

  • Please register or sign in to reply
    Loading