-
Notifications
You must be signed in to change notification settings - Fork 185
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
Upgrade Junction to Python 3.10 & Django 3.2 along with Social Login Fix #744
Merged
inovizz
merged 4 commits into
pythonindia:feature/upgrade-junction
from
Bhandari423:python-and-dependencies-upgrade
Jan 10, 2023
Merged
Upgrade Junction to Python 3.10 & Django 3.2 along with Social Login Fix #744
inovizz
merged 4 commits into
pythonindia:feature/upgrade-junction
from
Bhandari423:python-and-dependencies-upgrade
Jan 10, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bhandari423
changed the title
Sign in through social accounts
feature/upgrade-junction-epam
Nov 29, 2022
inovizz
changed the title
feature/upgrade-junction-epam
Upgrade Junction to Python 3.10 & Django 3.2 along with Social Login Fix
Dec 5, 2022
Bhandari423
force-pushed
the
python-and-dependencies-upgrade
branch
from
December 5, 2022 09:18
b9473fe
to
bc4c5f4
Compare
inovizz
reviewed
Jan 10, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
First of all great work @Bhandari423 and @rohitchopra-epam. Most of the changes are good, except for some small changes and clarifications.
inovizz
approved these changes
Jan 10, 2023
ananyo2012
added a commit
that referenced
this pull request
Jun 15, 2023
* Upgrade Junction to Python 3.10 & Django 3.2 along with Social Login Fix (#744) * Upgrade Python Version to 3.10 and Django Version 3.2 * Fix Github and Google Login/Signup * new package added to the requirements file * using f-strings Co-authored-by: @rohitchopra-epam * Fix unit tests add gunicorn (#748) * Fix failing tests, Add gunicorn Added nox method for running gunicorn * Pin dependencies --------- Co-authored-by: Ananya Maiti <Ananya_Maiti@epam.com> * Dockerize Junction (#749) * Initial commit for Dockerization * Fix review comments and dockerignore * Update dev.py.sample with runsslserver * Fixes for using default settings module * Remove Dockerfile.celery and use image from junction web image * Update docker-compose.test.yml to not depend on postgres db * Add static asset compilation in Docker image * Add docker-compose.prod.yml and update server port configuration in application * Add social oauth env vars --------- Co-authored-by: Sanchit Balchandani <balchandani.sanchit@gmail.com> Co-authored-by: Ananya Maiti <Ananya_Maiti@epam.com> * Add SITE_PREFIX env variable for site url prefix * Fix smtp setup for sending verification emails * Fix conference moderator filter * Fix template params fro absolute_url and is_proposal_reviewer * Add Django streamhandler logging with DEBUG (#763) * Fix proposal comment template parameters Fix userprofile dashboard url * Add restart:always to containers * Fix comment creation error caused sparingly Add review_comments=False for default comment view for logged in user * Add password for Redis DB * Fixes #765 Sort ProposalSection in descending order of creation date in ProposalSectionReviewer Add form * Add autocomplete_field for Proposal Reviewer to enable searcheable dropdown in add form * Update DEFAULT_FROM_EMAIL * Update common.py * Added username field in edit profile form (Issue-769) (#771) Co-authored-by: Rajat Rajdeep <rajat.rajdeep@avasant.com> * Update Devlopment setup docs (#773) * Update Devlopment setup docs * Instructions for setting up junction using Docker post junction upgrade. (#761) * Added instructions in the ReadMe file for setting up junction using Docker post junction upgrade. * Updated the psycopg version to resolve - SCRAM authentication requires libpq version 10 issue. --------- Co-authored-by: Rajat Rajdeep <rajat.rajdeep@avasant.com> --------- Co-authored-by: Ananya Maiti <Ananya_Maiti@epam.com> Co-authored-by: Rajat Rajdeep <46029666+RajatRajdeep@users.noreply.github.com> Co-authored-by: Rajat Rajdeep <rajat.rajdeep@avasant.com> * Rename travis yml * Remove status badges from README --------- Co-authored-by: Bhandari423 <69859505+Bhandari423@users.noreply.github.com> Co-authored-by: Ananya Maiti <Ananya_Maiti@epam.com> Co-authored-by: Sanchit Balchandani <balchandani.sanchit@gmail.com> Co-authored-by: Rajat Rajdeep <46029666+RajatRajdeep@users.noreply.github.com> Co-authored-by: Rajat Rajdeep <rajat.rajdeep@avasant.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This upgrade is going to be incremental, this PR fixes initial syntactical issues, dependencies versions and few features which are not present. The idea is to fix things incrementally in a separate branch and then once everything is stable and we perform regression testing and raise a final PR towards the main.
This merge request contains changes as per Python 3.10: