Skip to content

Conversation

@cp-at-mit
Copy link
Contributor

@cp-at-mit cp-at-mit commented Nov 19, 2025

What are the relevant tickets?

https://github.com/mitodl/hq/issues/8413

Description (What does it do?)

This Pr does a few things:

  1. Adds back the sycnhronization call to hubspot to sync all properties for whenever a user receives a certificate. This was removed here: Remove configure hubspot command #2667
  2. Adds a try/catch around the hubspot properties and user sync in the handle_create_course_run_certificate signal.
  3. Adds some code to remove any semi-colons from course/program titles/readable_ids. I see at least one course which has a semi colon in the title in Production. Semi colons are used by Hubspot to indicate multiple items in a list. The new code will remove semi-colons and hopefully resolve the issue of syncing user's with course/program certificates to Hubspot.

How can this be tested?

There isn't a quick and easy way to test this locally without setting up your own Hubspot environment. To test the semi-colon removal you can try running the configure_hubspot_properties management command with and without a course that contains a semi-colon in the title.

@github-actions
Copy link

OpenAPI Changes

Show/hide ## Changes for v0.yaml:
## Changes for v0.yaml:


## Changes for v1.yaml:


## Changes for v2.yaml:


Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

@cp-at-mit cp-at-mit marked this pull request as ready for review November 20, 2025 17:10
@annagav annagav self-requested a review November 24, 2025 20:04
Copy link
Contributor

@annagav annagav left a comment

Choose a reason for hiding this comment

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

👍

@cp-at-mit cp-at-mit merged commit 6ddb7ee into main Nov 26, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants