Skip to content

Conversation

@ChristopherChudzicki
Copy link
Contributor

@ChristopherChudzicki ChristopherChudzicki commented Jul 11, 2024

What are the relevant tickets?

Description (What does it do?)

Reverts a change from #1198... XPRO_PLATFORM_TRANSFORM keys should not have been updated.

How can this be tested?

  1. Locally on main, run ./manage.py backpopulate_xpro_data and check /api/v1/courses?platform=xpro. There should be none. (bug)
  2. On this branch, run ./manage.py backpopulate_xpro_data. You should now have several courses.

@ChristopherChudzicki ChristopherChudzicki added the Needs Review An open Pull Request that is ready for review label Jul 11, 2024
@ChristopherChudzicki ChristopherChudzicki marked this pull request as ready for review July 11, 2024 17:40
@mbertrand mbertrand self-assigned this Jul 11, 2024
Copy link
Member

@mbertrand mbertrand left a comment

Choose a reason for hiding this comment

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

👍

@ChristopherChudzicki ChristopherChudzicki merged commit a324ca9 into main Jul 11, 2024
This was referenced Jul 11, 2024
@rhysyngsun rhysyngsun deleted the cc/fix-xpro-etl branch February 7, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review An open Pull Request that is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants