search

edx / upgrades

Coordination and support for major Open edX technology upgrades
Apache License 2.0
0 stars 0 forks source link

Replace rate-xblock dependency #72

Closed Jawayria closed 2 years ago

Jawayria commented 3 years ago

We use the package rate-xblock in edx/edx-platform. This package is obsolete and deprecated. Please follow the guidance in https://openedx.atlassian.net/wiki/spaces/AC/pages/3036972032/Handling+Outdated+Dependencies to resolve the problem this poses for the Open edX Django 3.2 upgrade.

mariajgrimaldi commented 3 years ago

I opened a PR with the intent to give support for Django 3.2. Let's see!

natabene commented 3 years ago

@mariajgrimaldi Thanks so much for picking this up!

mariajgrimaldi commented 3 years ago

The master branch now supports Django 3.2. I asked for a new release. Let's wait and see

natabene commented 3 years ago

@mariajgrimaldi Wow, that was quick, thanks for an update!

mariajgrimaldi commented 3 years ago

I forgot to say that I just added tests for Django 3.2, and they passed, so the current release already supports that version :100:

natabene commented 3 years ago

@mariajgrimaldi Thanks so much for an update. Hopefully a release will follow soon.

jmbowman commented 2 years ago

Afraid there was some confusion here, we're actually using a fork of this in the openedx GitHub org: https://github.com/openedx/RateXBlock . The PR above should definitely help, I'm hoping it's a simple matter of applying those changes to our fork. https://openedx.atlassian.net/browse/BOM-3200 should cover that, so I'll go ahead and close this issue.