Migrate arkindex.dataimport frontend

Breaking changes

The callback URI has been moved to the API. OAuth authentications will stop working; GitLab will return "Invalid URI" errors.
Head over to https://gitlab.com/profile/applications and reconfigure the Arkindex applications' callback URIs.
- Before:
/imports/oauth/gitlab/callback
- After:
/api/v1/oauth/providers/gitlab/callback/
(do not forget the trailing slash, GitLab is slash-sensitive)
Edited by Erwan Rouchet
Merge request reports
Activity
mentioned in merge request frontend!61 (merged)
Codecov Report
Merging #162 into master will increase coverage by
0.12%
. The diff coverage is95.55%
.@@ Coverage Diff @@ ## master #162 +/- ## ========================================= + Coverage 72.68% 72.8% +0.12% ========================================= Files 179 177 -2 Lines 7890 7881 -9 ========================================= + Hits 5735 5738 +3 + Misses 2155 2143 -12
Impacted Files Coverage Δ arkindex/project/api_v1.py 100% <ø> (ø)
arkindex/users/providers.py 57.14% <ø> (ø)
arkindex/dataimport/urls.py 100% <100%> (ø)
arkindex/users/tests/test_providers.py 100% <100%> (ø)
arkindex/users/tests/test_gitlab_oauth.py 100% <100%> (ø)
arkindex/users/api.py 75.75% <88.23%> (+1.84%)
arkindex/documents/models.py 40.63% <0%> (-0.36%)
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update fd716d1...05b5e67. Read the comment docs.Edited by Bastien Abadieadded 6 commits
-
0c2c266a...fd716d15 - 2 commits from branch
master
- 163d8393 - Use FrontendView for arkindex.dataimport
- c7f14400 - Remove views and templates
- 5b89f10d - Move OAuth callback to API
- 961fa07b - Fix unit tests
Toggle commit list-
0c2c266a...fd716d15 - 2 commits from branch
added 1 commit
- 05b5e67e - Add a test for OAuth callback error handling
mentioned in commit 355ca569
Please register or sign in to reply