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

[SPIKE] Investigate the effort involved in upgrading to django 4 #1647

Closed
5 tasks
riatzukiza opened this issue Feb 18, 2022 · 5 comments · May be fixed by #2977
Closed
5 tasks

[SPIKE] Investigate the effort involved in upgrading to django 4 #1647

riatzukiza opened this issue Feb 18, 2022 · 5 comments · May be fixed by #2977
Labels
backend dev DUPE possible duplicate Refined Ticket has been refined at the backlog refinement spike techdebt

Comments

@riatzukiza
Copy link

riatzukiza commented Feb 18, 2022

Description:
When we initially tried updating to Django 4, we had a few things break and we want to find out how deep it might go before committing to any specific effort on it.

Acceptance Criteria:
Create a list of functional outcomes that must be achieved to complete this issue

  • We have a report detailing what is preventing tdrs from working with django 4
  • If more work is required, create an issue for actually upgrading, otherwise just do the upgrade

Tasks:
Create a list of granular, specific work items that must be completed to deliver the desired outcomes of this issue

  • investigate the errors in notes
  • find relavent documentation
  • compile findings into report (probably ADR?)

Notes:
Add additional useful information, such as related issues and functionality that isn't covered by this specific issue, and other considerations that will be helpful for anyone reading this

  • tdrs-backend-web-1 | django.core.exceptions.ImproperlyConfigured: 'colorfield' not needed since django 4.0, remove it from settings.INSTALLED_APPS.
  • tdrs-backend-web-1 | ModuleNotFoundError: No module named 'pytz'

Supporting Documentation:
Please include any relevant log snippets/files/screen shots

  • Doc 1
  • Doc 2

Open Questions:
Please include any questions or decisions that must be made before beginning work or to confidently call this issue complete

How to point a walk in the shadows of a rabbit hole?

@riatzukiza riatzukiza added the dev label Feb 18, 2022
@valcollignon
Copy link

valcollignon commented Feb 22, 2022

@abottoms-coder to refine further.

EDIT: After discussion with @riatzukiza , dev team needs to huddle to shed some light on this ticket.

Relevant link: https://github.com/raft-tech/django-admin-508/issues/3

@riatzukiza
Copy link
Author

riatzukiza commented Feb 22, 2022

tdrs-backend-web-1 | django.core.exceptions.ImproperlyConfigured: 'colorfield' not needed since django 4.0, remove it from settings.INSTALLED_APPS.

I removed colorfields, then we got:
tdrs-backend-web-1 | ModuleNotFoundError: No module named 'pytz'

I could not find in either django-admin-508 or tdrs-backend a reference to a module pytz.

@ADPennington
Copy link
Collaborator

discussed on 2/23 dev sync. this is an enhancement to upgrade to django v4. for tdp, this is low priority. still in unrefined state. cc: @abottoms-coder

@andrew-jameson andrew-jameson removed their assignment Mar 8, 2022
@stevenino stevenino changed the title [WIP] [SPIKE] Investigate the effort involved in upgrading to django 4 [SPIKE] Investigate the effort involved in upgrading to django 4 Sep 27, 2022
@stevenino stevenino added Refined Ticket has been refined at the backlog refinement backend labels Sep 27, 2022
@jtimpe jtimpe added the spike label Apr 12, 2024
@robgendron robgendron added the Old label Apr 16, 2024
@elipe17 elipe17 mentioned this issue May 8, 2024
28 tasks
@robgendron robgendron removed the Old label May 30, 2024
@robgendron robgendron added the DUPE possible duplicate label Jul 16, 2024
@robgendron
Copy link

DUPE - PR #2977 does this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend dev DUPE possible duplicate Refined Ticket has been refined at the backlog refinement spike techdebt
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants