Skip to content

Commit

Permalink
Merge pull request #634 from SUI-Components/feat/events-and-fces
Browse files Browse the repository at this point in the history
feat(components/login/passwordReset): Add events for login
  • Loading branch information
davidmartin84 committed Nov 14, 2023
2 parents 059aada + 30c00a9 commit 86763cb
Show file tree
Hide file tree
Showing 5 changed files with 34 additions and 22 deletions.
Original file line number Diff line number Diff line change
@@ -1,13 +1,12 @@
import PropTypes from 'prop-types'

import {BASE_CLASS} from '../../config.js'

import LoginForm from '../Form/LoginForm.js'
import FaceRecognition from '../Info/FaceRecognition.js'
import Form from './Form.js'
import LoginInfo from './LoginInfo.js'

const Login = ({icons}) => {
const Login = ({hasFaceRecognition = false, icons}) => {
return (
<div className={`${BASE_CLASS} no-padding`}>
<div className={`${BASE_CLASS}-item`}>
Expand All @@ -17,7 +16,7 @@ const Login = ({icons}) => {
<Form icons={icons} isLogin>
<>
<LoginForm icons={icons} />
<FaceRecognition icons={icons} />
{hasFaceRecognition && <FaceRecognition icons={icons} />}
</>
</Form>
</div>
Expand All @@ -28,7 +27,8 @@ const Login = ({icons}) => {
Login.displayName = 'Login'

Login.propTypes = {
icons: PropTypes.arrayOf(PropTypes.object)
icons: PropTypes.arrayOf(PropTypes.object),
hasFaceRecognition: PropTypes.bool
}

export default Login
20 changes: 8 additions & 12 deletions components/login/passwordReset/src/components/Form/LoginForm.js
Original file line number Diff line number Diff line change
@@ -1,12 +1,11 @@
import MoleculeInputField from '@s-ui/react-molecule-input-field'

import {BASE_CLASS} from '../../config.js'
import {BASE_CLASS, EVENTS} from '../../config.js'
import useLoginFormState from '../../hooks/components/useLoginFormState.js'
// import useDomain from '../../hooks/useDomain.js'
// import useEventBus from '../../hooks/useEventBus.js'
import useEventBus from '../../hooks/useEventBus.js'
import useI18n from '../../hooks/useI18n.js'
import Notification from '../Info/Notification.js'

// import LoginButton from '../Input/LoginButton.js'
import SubmitButton from '../Input/SubmitButton.js'

Expand All @@ -29,15 +28,9 @@ const LoginForm = () => {

const i18n = useI18n()
// const domain = useDomain()
// const {emit} = useEventBus()
const {emit} = useEventBus()

// const {
// RESET_PASSWORD_BUTTON_CLICK,
// RESET_PASSWORD_EMAIL_VALIDATION_ERROR,
// RESET_PASSWORD_ERROR,
// RESET_PASSWORD_RESEND_CLICK,
// RESET_PASSWORD_SUCCESS
// } = EVENTS
const {LOGIN} = EVENTS

// const getErrorText = value => {
// return domain
Expand Down Expand Up @@ -93,7 +86,10 @@ const LoginForm = () => {
// }

const handleSubmit = () => {
console.log('Llamada al login')
emit(LOGIN, {
email,
password
})
// emit(RESET_PASSWORD_BUTTON_CLICK, {email})
// getErrorText(email).then(errorText => {
// if (errorText) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,22 @@ import PropTypes from 'prop-types'

import AtomIcon, {ATOM_ICON_SIZES} from '@s-ui/react-atom-icon'

import {BASE_CLASS, DEFAULT_FACE_RECOGNITION_ICON} from '../../config.js'
import {
BASE_CLASS,
DEFAULT_FACE_RECOGNITION_ICON,
EVENTS
} from '../../config.js'
import useEventBus from '../../hooks/useEventBus.js'
import useI18n from '../../hooks/useI18n.js'
const FaceRecognition = ({icons}) => {
const {emit} = useEventBus()
const i18n = useI18n()

return (
<div className={`${BASE_CLASS}-formFooter ${BASE_CLASS}-formFooterIcon`}>
<div
className={`${BASE_CLASS}-formFooter ${BASE_CLASS}-formFooterIcon`}
onClick={() => emit(EVENTS.BIOMETRIC_LOGIN, null)}
>
<AtomIcon size={ATOM_ICON_SIZES.medium}>
{icons.faceRecognition || DEFAULT_FACE_RECOGNITION_ICON}
</AtomIcon>
Expand Down
4 changes: 3 additions & 1 deletion components/login/passwordReset/src/config.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 7 additions & 3 deletions components/login/passwordReset/src/index.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,18 @@
import PropTypes from 'prop-types'

import PasswordReset from './components/Content/PasswordReset.js'
import Login from './components/Content/Login.js'
import PasswordReset from './components/Content/PasswordReset.js'
import {STAGE_PASSWORD_CHANGE, STAGE_PASSWORD_RESET_START} from './config.js'
import {PasswordResetProvider} from './context.js'

export default function LoginPasswordReset(props) {
return (
<PasswordResetProvider {...props}>
{props.isLogin ? (
<Login icons={props.icons || {}} />
<Login
icons={props.icons || {}}
hasFaceRecognition={props.hasFaceRecognition}
/>
) : (
<PasswordReset icons={props.icons || {}} />
)}
Expand All @@ -36,5 +39,6 @@ LoginPasswordReset.propTypes = {
onEvent: PropTypes.func,
/* Allows to override the icons used in the component */
icons: PropTypes.arrayOf(PropTypes.object),
isLogin: PropTypes.bool
isLogin: PropTypes.bool,
hasFaceRecognition: PropTypes.bool
}

0 comments on commit 86763cb

Please sign in to comment.