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
SHARE-350 Remove unused OAuth stuff #782
SHARE-350 Remove unused OAuth stuff #782
Conversation
-removed unused old OAuth stuff from register.js -removed unused old OAuth stuff from templates -removed old OAuth test cases -removed old OAuth translations
Codecov Report
@@ Coverage Diff @@
## develop #782 +/- ##
=============================================
- Coverage 40.19% 37.25% -2.95%
- Complexity 4765 4847 +82
=============================================
Files 559 566 +7
Lines 14702 14999 +297
=============================================
- Hits 5910 5588 -322
- Misses 8792 9411 +619
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@@ -1,4 +1,4 @@ | |||
{# needed for Google Sign #} | |||
{# needed for OAuth Sign in #} | |||
<head> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
have you checked if we still need this font?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added that font because of the google branding guidelines. So that should be fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rest looks good to me!
-removed unused old OAuth stuff from register.js
-removed unused old OAuth stuff from templates
-removed old OAuth test cases
-removed old OAuth translations
Your checklist for this pull request
Please review the contributing guidelines and wiki pages of this repository.
SHARE-666 The devils ticket
Code Review
section in JiraAdditional Description
TODO: Add additional information that is not in your commit-message here
Tests - additional information
TODO: add additional information about testruns here