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 new 2 #245

Merged
merged 100 commits into from
May 3, 2024
Merged

Django upgrade new 2 #245

merged 100 commits into from
May 3, 2024

Conversation

David-Strupinski
Copy link
Contributor

No description provided.

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
abztrakt and others added 25 commits November 21, 2023 14:45
@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 @abztrakt
update license headers
6af6f45
@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
@David-Strupinski
use modified django-oauth-plus
3abb7b5
@David-Strupinski
Merge branch 'develop' into django-upgrade-new
dc46f45
@abztrakt
String instead of bool
34612e9
@David-Strupinski
try to fix migration issues, fully remove old oauth_provider
87a54e5
@David-Strupinski
fix style :(
7604441
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8944525129

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 8928287823: 0.3%
Covered Lines: 6385
Relevant Lines: 6603
💛 - Coveralls

@abztrakt abztrakt merged commit f89f528 into feature/eval-me May 3, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abztrakt abztrakt abztrakt approved these changes

@nhuang131 nhuang131 Awaiting requested review from nhuang131

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
@David-Strupinski @coveralls @abztrakt