Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

Sliced the description to exclude the section tags#291

Merged
mehallhm merged 1 commit intomasterfrom
html_tags_in_course_desc
Mar 27, 2025
Merged

Sliced the description to exclude the section tags#291
mehallhm merged 1 commit intomasterfrom
html_tags_in_course_desc

Conversation

@WesleyTran0
Copy link
Contributor

Purpose

Removed the weird tags in the course description from the description in the backend.

Tickets

#239

Contributors

@WesleyTran0
@mehallhm Thank you for the help <3

Feature List

  • sliced the decoded message pulled from banner
  • added a comment explaining what was sliced out

image
image

Reviewers

Primary reviewer:

  • Pick one primary reviewer
    • The team member with the most relevant knowledge about the code area this PR touches
    • NOT an author of the PR
    • If the primary reviewer is the project lead, select two primary reviewers
      • Goal: facilitate knowledge transfer to other team members
    • Primary reviewers are required to approve the PR before it can be merged

Primary:

Use the "Reviewers" feature in Github
@mehallhm @ananyaspatil

Secondary reviewers:

@ItsEricSun @cherman23 @nickpfeiffer05

Secondary:

@WesleyTran0 WesleyTran0 self-assigned this Mar 27, 2025
@coveralls
Copy link
Collaborator

Coverage Status

coverage: 79.716% (-0.03%) from 79.75%
when pulling d337717 on html_tags_in_course_desc
into bbf63ba on master.

@mehallhm mehallhm merged commit 8bc2689 into master Mar 27, 2025
9 checks passed
@mehallhm mehallhm deleted the html_tags_in_course_desc branch March 27, 2025 23:49
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