Use overwrite=True when calling pluggy function from upsert_offered_by

[mbertrand]

What are the relevant tickets?

• Closes https://github.com/mitodl/hq/issues/4763
• Closes https://github.com/mitodl/hq/issues/4761

Description (What does it do?)

• Adds the overwrite=True kwarg to the offeror_upserted_actions call within the upsert_offered_by_data function.
• Updates the upsert_offered_by_data function to use code instead of name as the primary key to use for upserts.

How can this be tested?

• Run the following if you haven't already done so:

  ./manage.py upsert_offered_by
  ./manage.py backpopulate_resource_channels
  

• Go to http://open.odl.local:8062/c/unit/mitx/ and verify that the title in the browser tab is the same as what's specified for MITx in the file learning_resources/fixtures/offered_by.json
• Change the name for mitx in that json file to something else like "MITx Offeror"
• Rerun ./manage.py update_offered_by
• Reload the url above, the browser tab title should now display the new name.

[shanbady]

The changes here look good and works as needed. Going to approve but i have a suggestion that likely warrants its own ticket. Since we now have the data_fixtures app, we should move our static fixture loading into the common app since folks will likely end up running this command after the data fixtures app migrates and theres no easy way to tell which one is the source of truth

[mbertrand]

Since we now have the data_fixtures app, we should move our static fixture loading into the common app since folks will likely end up running this command after the data fixtures app migrates and theres no easy way to tell which one is the source of truth

Do you plan to make an issue for that or should I?