Skip to content
This repository was archived by the owner on Feb 22, 2024. It is now read-only.

Commit bbcdcc7

Browse files
authored
yashim/fix: downgrade onfido version (#6574)
* fix: downgrade onfido version * retrigger circle-ci * reverted package lock * fix: ammended comments from code review
1 parent b95f473 commit bbcdcc7

File tree

4 files changed

+85
-69
lines changed

4 files changed

+85
-69
lines changed

package-lock.json

Lines changed: 78 additions & 54 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@
128128
"js-cookie": "2.2.0",
129129
"kinetic": "5.2.0",
130130
"moment": "2.19.4",
131-
"onfido-sdk-ui": "^6.7.2",
131+
"onfido-sdk-ui": "^6.3.0",
132132
"promise-polyfill": "6.1.0",
133133
"prop-types": "15.6.1",
134134
"react": "16.8.6",

src/javascript/app/pages/user/account/authenticate.js

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -881,15 +881,9 @@ const Authenticate = (() => {
881881
country: country_code,
882882
} : false,
883883
},
884-
useLiveDocumentCapture: true,
885-
},
886-
},
887-
{
888-
type : 'face',
889-
options: {
890-
useLiveDocumentCapture: true,
891884
},
892885
},
886+
'face',
893887
],
894888
});
895889
$('#authentication_loading').setVisibility(0);

src/sass/static/pages.scss

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -959,11 +959,8 @@ body #not_authenticated_financial ul.checked > li {
959959
.onfido-sdk-ui-NavigationBar-back {
960960
background: transparent;
961961
}
962-
.onfido-sdk-ui-crossDevice-CrossDeviceLink-numberInputSection {
963-
.ods-button.-action--primary {
964-
height: 100%;
965-
font-size: 0.875em;
966-
}
962+
.onfido-sdk-ui-Button-button-primary:hover {
963+
background: rgb(53, 63, 244); // Onfido's primary color
967964
}
968965
.onfido-sdk-ui-Button-button-outline:hover {
969966
background-color: transparent;
@@ -973,6 +970,7 @@ body #not_authenticated_financial ul.checked > li {
973970
color: $COLOR_BLACK;
974971
border-radius: 0;
975972
}
973+
/** TODO: To re-implement style overrides when upgrading Onfido version
976974
.onfido-sdk-ui-Theme-textButton:hover {
977975
color: $COLOR_BLACK;
978976
}
@@ -987,14 +985,14 @@ body #not_authenticated_financial ul.checked > li {
987985
background: rgb(92, 108, 255);
988986
}
989987
}
990-
}
988+
} **/
991989
input, textarea {
992990
&:hover, &:focus {
993991
outline: 0;
994992
border: none;
995993
}
996994
}
997-
button:not(.onfido-sdk-ui-Theme-link):not(.ods-button) {
995+
button:not(.onfido-sdk-ui-Theme-link):not(.onfido-sdk-ui-Button-button-primary) {
998996
&:hover, &:focus {
999997
background-color: inherit;
1000998
}

0 commit comments

Comments
 (0)