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

BOM-1079 Django22 Support #47

Merged
merged 1 commit into from Dec 6, 2019
Merged

BOM-1079 Django22 Support #47

merged 1 commit into from Dec 6, 2019

Conversation

Ayub-Khan
Copy link
Contributor

@Ayub-Khan Ayub-Khan commented Dec 2, 2019

Description: Describe in a couple of sentence what this PR adds
Adds Django 22 support
-updated tox and travis configs
-updated requirements
-updated setup file

JIRA: Link to JIRA ticket
https://openedx.atlassian.net/browse/BOM-1079

Dependencies: dependencies on other outstanding PRs, issues, etc.
None so far.

Reviewers:

  • tag reviewer

Merge checklist:

  • All reviewers approved
  • CI build is green
  • Version bumped
  • Changelog record added
  • Commits are squashed

Post merge:

  • Create a tag
  • Check new version is pushed to PyPI after tag-triggered build is
    finished.
  • Delete working branch (if not needed anymore)

@Ayub-Khan Ayub-Khan changed the title (wip) BOM-1079 Django22 Support BOM-1079 Django22 Support Dec 2, 2019
edx_ace/channel/sailthru.py Outdated Show resolved Hide resolved
edx_ace/message.py Outdated Show resolved Hide resolved
edx_ace/message.py Outdated Show resolved Hide resolved
requirements/travis.in Outdated Show resolved Hide resolved
@Ayub-Khan Ayub-Khan force-pushed the BOM-1079-Django22-Support branch 2 times, most recently from b9e1c38 to 2009365 Compare December 6, 2019 09:05
-updated tox and travis configs
-updated requirements
-updated setup file
-fixed tests
-updated trove classifiers
-include deprication warnings
-fixed quality
@Ayub-Khan Ayub-Khan merged commit 6423d08 into master Dec 6, 2019
@Ayub-Khan Ayub-Khan deleted the BOM-1079-Django22-Support branch December 6, 2019 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants