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

Feature/eval me #253

Merged
merged 121 commits into from
Mar 7, 2025
Merged

Feature/eval me #253

merged 121 commits into from
Mar 7, 2025

Conversation

abztrakt
Copy link
Member

@abztrakt abztrakt commented Mar 7, 2025

Testing moving test.api to django 2 AND postgres

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 28 commits April 2, 2024 14:14
@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
@abztrakt
Merge pull request #245 from uw-it-aca/django-upgrade-new-2
f89f528 
Django upgrade new 2
@abztrakt
Move to scout-v2
98d56a5
@abztrakt
Use django container 1.4.2
cbeb8b8
@abztrakt
A couple of PEP8 fixes
5d8ca38
@abztrakt
isinstance() over ==, per pycodestyle
bb3e594
@abztrakt
Update django-container version to latest
05f4106
@abztrakt
Update github action
aeff017
@abztrakt
Appease the GitHub warning
b16f1a4
@abztrakt
Merge pull request #246 from uw-it-aca/postgres
c490026 
Move to scout-v2
@abztrakt
Update to use postgres client
fbe7e52
@abztrakt
psychopg2 was needed after all
fa80507
@abztrakt
Merge branch 'feature/eval-me' into mysql-to-postgresv2
d24430f
@abztrakt
Merge pull request #248 from uw-it-aca/mysql-to-postgresv2
875bd99 
psychopg2 was needed after all
@abztrakt
Really just a commit to trigger github action run
bd28d49
@abztrakt
Revert "Really just a commit to trigger github action run"
f44e822 
This reverts commit bd28d49.
@abztrakt
Internal IP
f01e021
@abztrakt
Use the correct VPC
1638f8e
@abztrakt
Start with fresh migration
03ecb81
@abztrakt
Merge pull request #250 from uw-it-aca/new-migration
1ba06fc 
New migration
@abztrakt
Update test values for postgres db
ef034ee
@abztrakt
Merge pull request #251 from uw-it-aca/django-upgrade
aec8dbe 
eval-me <- Django upgrade
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13731137812

Details

  • 332 of 337 (98.52%) changed or added relevant lines in 57 files are covered.
  • 5 unchanged lines in 3 files lost coverage.
  • Overall coverage increased (+0.3%) to 96.72%
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/org_forms/uiuc_spot.py 1 0.0%
spotseeker_server/views/person.py 1 50.0%
spotseeker_server/logger/oauth.py 3 89.29%
Totals Coverage Status
Change from base Build 8562675062: 0.3%
Covered Lines: 6457
Relevant Lines: 6676
💛 - Coveralls

@abztrakt abztrakt merged commit 7392007 into develop Mar 7, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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