Skip to content
This repository has been archived by the owner on Apr 24, 2024. It is now read-only.

Adds related course pages #564

Merged
merged 5 commits into from
Jan 4, 2024
Merged

Adds related course pages #564

merged 5 commits into from
Jan 4, 2024

Conversation

kevinhowbrook
Copy link
Contributor

@kevinhowbrook kevinhowbrook commented Jan 3, 2024

⚠️ based on #562

Description of Changes Made

This PR adds the related course page model, so courses can be related to one another.

How to Test

  1. Check out this branch locally
  2. Add a course landing page
  3. Add 3 course pages
  4. Relate some course pages to eachother
  5. View front end of course page and you should see the related course pages (unstyled)

MR Checklist

  • Add a description of your pull request and instructions for the reviewer to verify your work.
  • If your pull request is for a specific ticket, link to it in the description.
  • Stay on point and keep it small so the merge request can be easily reviewed.
  • Consider adding unit tests, especially for bug fixes. If you don't, tell us why.
  • Tests and linting passes.
  • Consider updating documentation. If you don't, tell us why.
  • If relevant, list the environments / browsers in which you tested your changes.

Base automatically changed from course-landing-page to courses January 4, 2024 11:21
Copy link
Contributor

@William-Blackie William-Blackie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, one suggestion :)

tbx/courses/models.py Outdated Show resolved Hide resolved
@kevinhowbrook kevinhowbrook merged commit 70aa231 into courses Jan 4, 2024
3 checks passed
@kevinhowbrook kevinhowbrook deleted the related-courses branch January 4, 2024 15:45
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.

3 participants