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

[DEV-10748] Fix for psycopg2.errors.InvalidCursorName while running tests #4052

Merged
merged 6 commits into from
Apr 19, 2024

Conversation

ayubshahab
Copy link
Contributor

@ayubshahab ayubshahab commented Mar 20, 2024

The newer version of Django was resulting in the psycopg2.errors.InvalidCursorName as certain functionality has been changed in newer versions. Viewing the documentation and stackoverflow links I realized that this is a known issue while upgrading to the latest version of Django. However, the version of Django we were previously using, 3.2.* the minor version is still compatible with python 3.10.13 so reverted back to that.

Description:
Reverted the Django dependency version back to 3.2.*.

Technical details:
The technical details for the knowledge of other developers. Any detailed caveats or specific deployment steps should be outlined here.

Requirements for PR merge:

  1. Unit & integration tests updated
  2. API documentation updated
  3. Necessary PR reviewers:
    • Backend
  4. Appropriate Operations ticket(s) created
  5. Jira Ticket DEV-10748:
    • Link to this Pull-Request

Area for explaining above N/A when needed:

…rrors.InvalidCursorName as seemingly certain functionality has been changed in newer versions. However, the version of Django we were previously using, 3.2.* the minor version is still compatible with python 3.10.13 so reverted back to that and the error resolved itself.
@ayubshahab ayubshahab added the do not merge [pr] This PR should not be merged label Mar 20, 2024
…n version 3.10.13 but so is django version 4.2.11. Testing this now to see if it causes any additional errors with existing functinality and tests.
Base automatically changed from dev-10805-black-reformatting to dev-10645-python-upgrade-3.10 March 20, 2024 20:28
@ayubshahab ayubshahab merged commit 566f27f into dev-10645-python-upgrade-3.10 Apr 19, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge [pr] This PR should not be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant