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.

Sign up for GitHub

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

Jump to bottom

Django upgrade #242

Merged
merged 94 commits into from
Apr 2, 2024
Merged

Django upgrade #242

merged 94 commits into from
Apr 2, 2024

Conversation

abztrakt
Copy link
Member

@abztrakt abztrakt commented Apr 2, 2024

  • Upgrade django from 1 to 2
  • Implement new supported oauth as oauth 2

David-Strupinski and others added 30 commits March 29, 2023 14:16
@David-Strupinski
comment out auth and deprecated tests
32e8d30
@David-Strupinski
disable auth
3b502f8
@David-Strupinski
fully remove auth
cb2ea85
@David-Strupinski
update to django 2.0.*
673b58c
@David-Strupinski
fix ForeignKey fields in models
a5bda63
@David-Strupinski
update url patterns to new syntax
983b4dd
@David-Strupinski
remove django prometheus
f59ada0
@David-Strupinski
upgrade to django 2.2.*
0be54bf
@David-Strupinski
add django-oauth-toolkit auth, tests don't pass
5141543
@David-Strupinski
add migrations back, problem with new Client persists
95707bd
@David-Strupinski
fix migration problem, tests still don't pass
3c8c3db
@David-Strupinski @abztrakt
remove Favorites
36758cf
@David-Strupinski @abztrakt
remove reviews, ratings
e3bd137
@David-Strupinski @abztrakt
remove shared spaces
aa590c9
@David-Strupinski @abztrakt
apply migrations
a7f1acd
@abztrakt
Seeing if this will even run on ubuntu 20.04
c9fb3be
@David-Strupinski @abztrakt
update copyright year
98a8be0
@David-Strupinski @abztrakt
update copyright on non-checked files
7c79a81
@abztrakt
Ubuntu 18 does not work anymore
41b347e
@abztrakt
Load spottypes
25ce5b4
@David-Strupinski
Merge branch 'django-upgrade' into task/oauth-change
1d54a48
@David-Strupinski
merge migrations
b2541db
@David-Strupinski
comment out last of old auth, passes testts
761c355
@David-Strupinski
remove user_auth_required comments
6444cf1
@David-Strupinski
update dummy fixture to use new client
3d8fefa
@David-Strupinski
ignore vscode files
ecf9299
@David-Strupinski
move admin site out of debug
fcc63b5
@David-Strupinski
remove old comment
8f8c668
@David-Strupinski
update client
e5ec3a5
@David-Strupinski
lowercase all REST calls
0eb6d4b
David-Strupinski and others added 26 commits November 14, 2023 12:50
@David-Strupinski
add extra techloan office to sample spots
fdacfc7
@David-Strupinski
add on_delete=models.CASCADE to all foreign keys
001aa64
@David-Strupinski
squash migrations
5910de8
@abztrakt
Restore dev-values.yml
7319e65
@abztrakt
Merge pull request #228 from uw-it-aca/task/prep-upgrade
2a00741 
Task/prep upgrade
@abztrakt
Re-add (again) dev-values and update cicd to use it
7965fdd
@abztrakt
Merge pull request #231 from uw-it-aca/task/prep-upgrade
b41422b 
Re-add (again) dev-values and update cicd to use it
@abztrakt
Merge branch 'feature/eval-me' into django-upgrade
ffc54bd
@abztrakt
Remove bad squashed migration
7648be1
@David-Strupinski
Revert "Merge branch 'feature/eval-me' into django-upgrade"
4a4862f 
This reverts commit ffc54bd, reversing
changes made to fdacfc7.
@abztrakt
Merge pull request #232 from uw-it-aca/django-upgrade
bc82cb2 
Django upgrade
@abztrakt
Add these guys again?!
015d573
@abztrakt
Fix prometheus 404s again?
9e0bd76
@abztrakt
Base settings have important stuff
9b679af
@abztrakt
Merge pull request #233 from uw-it-aca/feature/eval-me
0666a10 
Feature/eval me
@abztrakt
set secret refresh interval lower on dev
a37a584
@David-Strupinski @abztrakt
update UW copyright year
a2d306c
@abztrakt
refresh interval on the external, not environment secrets
1d70f65
@abztrakt
refresh interval on the external secret
10b6121
@abztrakt
Back to default refresh interval
d93840e
@David-Strupinski
fix migrations to let 6-8 work on develop
bf7f464
@David-Strupinski
simplify migrations
4f4c8f6
@David-Strupinski
remove unused imports
d5debda
@abztrakt
Merge pull request #239 from uw-it-aca/fix-migrations
cf1ec49 
fix migrations to let 6-8 work on develop
@David-Strupinski @abztrakt
update license headers and remove unneeded files
a486f8e
@abztrakt
Merge branch 'temp-develop' into django-upgrade
a21b1a3 
Removes unneeded files
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8529791248

Details

  • 331 of 336 (98.51%) changed or added relevant lines in 56 files are covered.
  • 4 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.3%) to 96.698%
Changes Missing Coverage Covered Lines Changed/Added Lines %
spotseeker_server/models/auth.py 12 13 92.31%
spotseeker_server/urls.py 5 7 71.43%
spotseeker_server/views/rest_dispatch.py 31 33 93.94%
Files with Coverage Reduction New Missed Lines %
spotseeker_server/views/person.py 1 50.0%
spotseeker_server/logger/oauth.py 3 89.29%
Totals Coverage Status
Change from base Build 8116466789: 0.3%
Covered Lines: 6385
Relevant Lines: 6603
💛 - Coveralls

@abztrakt abztrakt merged commit f0986a9 into develop Apr 2, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@David-Strupinski David-Strupinski David-Strupinski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@abztrakt @coveralls @David-Strupinski