Skip to content

remove DeprecatedEdxPlatformImportWarning - #29

Merged
nedbat merged 1 commit into
openedx:masterfrom
BbrSofiane:sof/squash_warnings
Nov 23, 2020
Merged

remove DeprecatedEdxPlatformImportWarning#29
nedbat merged 1 commit into
openedx:masterfrom
BbrSofiane:sof/squash_warnings

Conversation

@BbrSofiane

Copy link
Copy Markdown
Contributor

Remove warnings when from staff_graded when running:

manage.py lms createsuperuser

2020-11-23 14:28:11,450 WARNING 240810 [py.warnings] [user None] [ip None] warnings.py:109 - /edx/app/edxapp/venvs/edxapp/lib/python3.8/site-packages/staff_graded/staff_graded.py:27: DeprecatedEdxPlatformImportWarning: Importing course_modes instead of common.djangoapps.course_modes is deprecated
  from course_modes.models import CourseMode

@openedx-webhooks

Copy link
Copy Markdown

Thanks for the pull request, @BbrSofiane! I've created OSPR-5234 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

@openedx-webhooks openedx-webhooks added needs triage open-source-contribution PR author is not from Axim or 2U labels Nov 23, 2020
@nedbat
nedbat merged commit 8dac88c into openedx:master Nov 23, 2020
@openedx-webhooks

Copy link
Copy Markdown

@BbrSofiane 🎉 Your pull request was merged!

Please take a moment to answer a two question survey so we can improve your experience in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merged open-source-contribution PR author is not from Axim or 2U

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants