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.

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

Making HRD work in signup page and showing an error when HRD is detected in the forgot password page #1278

Merged
merged 2 commits into from
Mar 8, 2018
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion scripts/complete-translations.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,9 @@ const translateKey = async (toTranslate, lang) => {
.set('Content-Type', 'application/json')
.query({ tl: lang })
.query({ q: escapeWildCards(toTranslate) });
return restoreWildCards(result.body[0][0][0]);
const phrases = result.body[0].map(p => p[0]);
const singlePhrase = phrases.join('');
return restoreWildCards(singlePhrase);
};

const run = async () => {
Expand Down
19 changes: 19 additions & 0 deletions src/connection/database/reset_password.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,11 @@ import ResetPasswordPane from './reset_password_pane';
import { authWithUsername, hasScreen } from './index';
import { cancelResetPassword, resetPassword } from './actions';
import { renderPasswordResetConfirmation } from './password_reset_confirmation';
import { databaseUsernameValue } from '../../connection/database/index';
import { isHRDDomain } from '../../connection/enterprise';
import * as i18n from '../../i18n';
import * as l from '../../core/index';
import { swap, updateEntity } from '../../store/index';

const Component = ({ i18n, model }) => {
const headerText = i18n.html('forgotPasswordInstructions') || null;
Expand Down Expand Up @@ -36,6 +40,21 @@ export default class ResetPassword extends Screen {
getScreenTitle(m) {
return i18n.str(m, 'forgotPasswordTitle');
}
isSubmitDisabled(m) {
Copy link
Contributor

@lbalmaceda lbalmaceda Mar 8, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this used? Tests?

const tryingToResetPasswordWithHRDEmail = isHRDDomain(m, databaseUsernameValue(m));
if (tryingToResetPasswordWithHRDEmail) {
swap(
updateEntity,
'lock',
l.id(m),
l.setGlobalError,
i18n.str(m, ['error', 'forgotPassword', 'enterprise_email'])
);
} else {
swap(updateEntity, 'lock', l.id(m), l.clearGlobalError);
}
return tryingToResetPasswordWithHRDEmail;
}

submitHandler() {
return resetPassword;
Expand Down
7 changes: 6 additions & 1 deletion src/engine/classic/sign_up_screen.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@ import { hasOnlyClassicConnections, isSSOEnabled, useBigSocialButtons } from '..
import { renderSignedInConfirmation } from '../../core/signed_in_confirmation';
import { renderSignedUpConfirmation } from '../../connection/database/signed_up_confirmation';
import { renderOptionSelection } from '../../field/index';
import { logIn as enterpriseLogIn } from '../../connection/enterprise/actions';
import { logIn as enterpriseLogIn, startHRD } from '../../connection/enterprise/actions';
import { databaseUsernameValue } from '../../connection/database/index';
import { isHRDDomain } from '../../connection/enterprise';
import * as l from '../../core/index';
import * as i18n from '../../i18n';

Expand Down Expand Up @@ -85,6 +87,9 @@ export default class SignUp extends Screen {

submitHandler(m) {
if (hasOnlyClassicConnections(m, 'social')) return null;
if (isHRDDomain(m, databaseUsernameValue(m))) {
return id => startHRD(id, databaseUsernameValue(m));
}
if (isSSOEnabled(m)) return enterpriseLogIn;
return signUp;
}
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/af.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@ export default {
error: {
forgotPassword: {
too_many_requests: 'Jammer, jy het limiet bereik. Probeer asseblief weer later.',
'lock.fallback': 'Jammer, iets het verkeerd gegaan terwyl jy wagwoord verander het.'
'lock.fallback': 'Jammer, iets het verkeerd gegaan terwyl jy wagwoord verander het.',
enterprise_email:
"Jou e-pos se domein is deel van 'n Enterprise Identity provider. Om u wagwoord terug te stel, raadpleeg asseblief u sekuriteitsadministrateur."
},
login: {
blocked_user: 'Die gebruiker is geblok.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/ca.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ export default {
forgotPassword: {
too_many_requests:
"S'han exhaurit els intents per restablir la contrasenya. Espereu una estona i intenteu-ho de nou.",
'lock.fallback': 'Hi ha hagut un error en canviar la contrasenya.'
'lock.fallback': 'Hi ha hagut un error en canviar la contrasenya.',
enterprise_email:
"El domini del vostre correu electrònic forma part d'un proveïdor d'identitat empresarial. Per restablir la contrasenya, consulteu l'administrador de seguretat."
},
login: {
blocked_user: "L'usuari està bloquejat.",
Expand Down
9 changes: 5 additions & 4 deletions src/i18n/cs.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ export default {
forgotPassword: {
too_many_requests:
'Dosáhl jsi limitu pro počet pokusů o změnu hesla. Prosím počkej před dalším pokusem.',
'lock.fallback': 'Je nám líto, ale něco se pokazilo při změně hesla.'
'lock.fallback': 'Je nám líto, ale něco se pokazilo při změně hesla.',
enterprise_email:
'Doména vašeho e-mailu je součástí poskytovatele podnikové identity. Chcete-li obnovit heslo, obraťte se na správce zabezpečení.'
},
login: {
blocked_user: 'Uživatel je blokován.',
Expand All @@ -26,8 +28,7 @@ export default {
'Je třeba aktualizovat heslo, neboť jde o první přihlášení; mohla také vypršet platnost hesla.',
password_leaked:
'Zablokovali jsme uživatelský účet, protože došlo k možnému úniku hesla na jiné webové stránce. Poslali jsme instrukce k odblokování na email.',
too_many_attempts:
'Účet byl zablokován z důvodu příliš velkého počtu pokusů o přihlášení.',
too_many_attempts: 'Účet byl zablokován z důvodu příliš velkého počtu pokusů o přihlášení.',
session_missing:
'Nemohl jsem dokončit váš požadavek na ověření. Zkus to znovu po zavření všech otevřených dialogových oken.',
'hrd.not_matching_email': 'Prosím, použijte pro přihlášení firemní e-mail.'
Expand Down Expand Up @@ -123,5 +124,5 @@ export default {
mfaLoginTitle: 'Dvoufázové ověření',
mfaLoginInstructions: 'Prosím zadej ověřovací kód vygenerovaný mobilní aplikací.',
mfaSubmitLabel: 'Přihlásit',
mfaCodeErrorHint: 'Použijte %d číslic',
mfaCodeErrorHint: 'Použijte %d číslic'
};
4 changes: 3 additions & 1 deletion src/i18n/da.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ export default {
forgotPassword: {
too_many_requests:
'Du har nået grænsen for forsøg på at skifte adgangskode. Vent venligst før du prøver igen.',
'lock.fallback': 'Vi beklager, men der skete en fejl i forespørgslen efter ny adgangskode.'
'lock.fallback': 'Vi beklager, men der skete en fejl i forespørgslen efter ny adgangskode.',
enterprise_email:
'Dit e-mail-domæne er en del af en Enterprise Identity-udbyder. For at nulstille dit kodeord, se venligst din sikkerhedsadministrator.'
},
login: {
blocked_user: 'Denne bruger er blokeret.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/de.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ export default {
forgotPassword: {
too_many_requests:
'Sie haben das Limit für die Rücksetzung des Passworts erreicht. Bitte warten Sie, bevor Sie es erneut versuchen.',
'lock.fallback': 'Beim Zurücksetzen des Passworts ist ein Fehler aufgetreten.'
'lock.fallback': 'Beim Zurücksetzen des Passworts ist ein Fehler aufgetreten.',
enterprise_email:
'Die Domain Ihrer E-Mail ist Teil eines Enterprise Identity Providers. Um Ihr Passwort zurückzusetzen, wenden Sie sich bitte an Ihren Sicherheitsadministrator.'
},
login: {
blocked_user: 'Der Benutzer wird blockiert.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/en.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ export default {
forgotPassword: {
too_many_requests:
'You have reached the limit on password change attempts. Please wait before trying again.',
'lock.fallback': "We're sorry, something went wrong when requesting the password change."
'lock.fallback': "We're sorry, something went wrong when requesting the password change.",
enterprise_email:
"Your email's domain is part of an Enterprise identity provider. To reset your password, please see your security administrator."
},
login: {
blocked_user: 'The user is blocked.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/es.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ export default {
forgotPassword: {
too_many_requests:
'Se ha alcanzado el límite de intentos para restablecer su contraseña. Por favor, aguarde unos minutos.',
'lock.fallback': 'Ocurrió un error al restablecer su contraseña.'
'lock.fallback': 'Ocurrió un error al restablecer su contraseña.',
enterprise_email:
'El dominio de su correo electrónico es parte de un proveedor de identidad Enterprise. Para restablecer su contraseña, consulte a su administrador de seguridad.'
},
login: {
blocked_user: 'El usuario se encuentra bloqueado.',
Expand Down
10 changes: 7 additions & 3 deletions src/i18n/et.js
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
// This file was automatically translated.
// Feel free to submit a PR if you find a more accurate translation.

export default {
error: {
forgotPassword: {
too_many_requests:
'Sa oled liiga palju kordi üritanud salasõna vahetada. Palun oota enne uuesti proovimist.',
'lock.fallback': 'Vabandame, midagi läks salasõna vahetamise sooviga valesti.'
'lock.fallback': 'Vabandame, midagi läks salasõna vahetamise sooviga valesti.',
enterprise_email:
'Teie e-posti domeen kuulub ettevõtte identiteedi pakkuja juurde. Parooli lähtestamiseks lugege oma turvameedet.'
},
login: {
blocked_user: 'Kasutaja on blokeeritud.',
Expand All @@ -19,7 +24,7 @@ export default {
'Mitmetasemeline autentimine on nõutud aga sinu seade ei ole nimekirja lisatud. Palun lisa ta nimekirja.',
'lock.mfa_invalid_code': 'Vale kood. Palun proovi uuesti.',
password_change_required:
'Parooli vahetamine on kohustuslik, sest sa logid sisse esimest korda või parool on aegunud.', // TODO: verify error code
'Parooli vahetamine on kohustuslik, sest sa logid sisse esimest korda või parool on aegunud.',
password_leaked:
'Me oleme avastanud võimaliku turvariski selle kontoga. Sinu konto kaitsmises oleme selle sisselogimise blokeerinud. Sulle saadeti e-mail kuidas blokeering maha võtta.',
too_many_attempts: 'Sinu konto blokeeriti peale mitut ebaõnnestunud sisselogimiskatset.',
Expand All @@ -43,7 +48,6 @@ export default {
}
},
success: {
// success messages show above the form or in a confirmation pane
logIn: 'Täname, et sisse logisid.',
forgotPassword: 'Me saatsime sulle e-maili, et oma salasõna taastada.',
magicLink: 'Me saatsime sisselogimise lingi aadressile:<br />%s.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/fa.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,9 @@ export default {
forgotPassword: {
too_many_requests:
'بیش از دفعات مجاز تغییر رمز عبور تلاش نموده اید ، لطفا کمی صبر کنید و دوباره تلاش کنید',
'lock.fallback': 'متاسفیم ، مشکلی در تغییر رمز عبور رخ داده است.'
'lock.fallback': 'متاسفیم ، مشکلی در تغییر رمز عبور رخ داده است.',
enterprise_email:
'دامنه ایمیل شما بخشی از ارائهدهنده هویت سازمانی است. برای بازنشانی گذرواژه خود، لطفا به مدیر امنیتی خود مراجعه کنید.'
},
login: {
blocked_user: 'کاربر مسدود شده است.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/fi.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@ export default {
too_many_requests:
'Olet yrittänyt vaihtaa salasanaa liian monta kertaa. Ole hyvä ja odota ennen kuin yrität uudelleen.',
'lock.fallback':
'Olemme pahoillamme, mutta jotain meni vikaan kun salasanaa yritettiin vaihtaa.'
'Olemme pahoillamme, mutta jotain meni vikaan kun salasanaa yritettiin vaihtaa.',
enterprise_email:
'Sähköpostisi verkkotunnus on osa yrityspalvelun tarjoajaa. Voit palauttaa salasanasi turva-järjestelmänvalvojalta.'
},
login: {
blocked_user: 'Käyttäjä on estetty.',
Expand Down
4 changes: 3 additions & 1 deletion src/i18n/fr.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,9 @@ export default {
too_many_requests:
'Vous avez atteint la limite de tentatives de changement de mot de passe. Veuillez patienter avant de recommencer.',
'lock.fallback':
'Nous sommes désolés, un problème est survenu lors de la demande de changement de mot de passe.'
'Nous sommes désolés, un problème est survenu lors de la demande de changement de mot de passe.',
enterprise_email:
"Le domaine de votre messagerie fait partie d'un fournisseur d'identité d'entreprise. Pour réinitialiser votre mot de passe, veuillez contacter votre administrateur de sécurité."
},
login: {
blocked_user: 'L’utilisateur est bloqué.',
Expand Down
Loading