Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update or replace django-dynamic-fixture #26

Closed
Jawayria opened this issue Jul 27, 2021 · 12 comments
Closed

Update or replace django-dynamic-fixture #26

Jawayria opened this issue Jul 27, 2021 · 12 comments
Assignees
Labels
outdated dependency A dependency needs to be updated or replaced to support an upgrade initiative

Comments

@Jawayria
Copy link

We use the package django-dynamic-fixture in edx/blockstore, edx/demographics, edx/edx-analytics-dashboard, edx/edx-analytics-data-api, edx/edx-django-sites-extensions, edx/enterprise-catalog, edx/license-manager, edx/portal-designer, edx/registrar, edx/video-encode-manager. It hasn't yet added support for Django3.2. 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.

@Jawayria Jawayria added the outdated dependency A dependency needs to be updated or replaced to support an upgrade initiative label Jul 27, 2021
@cmltaWt0
Copy link

cmltaWt0 commented Sep 9, 2021

There is some work in progress regarding the Django 3.2 support.
paulocheque/django-dynamic-fixture#141

@natabene
Copy link

natabene commented Sep 9, 2021

@cmltaWt0 Thanks for finding this, I appreciate. Can you look into it further to help move things in time for Maple? Can I assign this to you?

@cmltaWt0
Copy link

@natabene Yes, please. Will take it.

@cmltaWt0
Copy link

paulocheque/django-dynamic-fixture#144

@cmltaWt0
Copy link

cmltaWt0 commented Oct 4, 2021

New version is live https://pypi.org/project/django-dynamic-fixture/3.1.2/
Going to create PRs to all required repos.

@cmltaWt0
Copy link

cmltaWt0 commented Oct 4, 2021

@natabene
I have no access to the following repos:

@natabene
Copy link

natabene commented Oct 5, 2021

@natabene I have no access to the following repos:

@Jawayria or @awais786 , please create BOM tickets for the 2 repos in question.

@Jawayria
Copy link
Author

We already have latest version of django-dynamics-fixture in demographics and I've merged the PR to upgrade the version in video-encode-manager as well

@natabene
Copy link

@Jawayria What is this left on this issue?

@Jawayria
Copy link
Author

@natabene just one PR openedx-unsupported/blockstore#130 is pending due to some conflicts. @cmltaWt0 please resolve the conflicts and let us know.

@cmltaWt0
Copy link

@Jawayria @natabene Closing due to PR #128 where the same upgrade was introduced. So all repos are upgraded.

@Jawayria
Copy link
Author

@natabene I guess this issue can be closed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
outdated dependency A dependency needs to be updated or replaced to support an upgrade initiative
Projects
None yet
Development

No branches or pull requests

3 participants