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

ADD a message for 3030 error #90

Merged
merged 2 commits into from
Jan 8, 2019
Merged

Conversation

new69
Copy link
Contributor

@new69 new69 commented Dec 3, 2018

Description

Create an error message for code 3030

Related Issues

Changes

  • Add new message for code 3030 in ErrorInterceptor.js;

@new69 new69 self-assigned this Dec 3, 2018
@@ -213,6 +213,11 @@ angular.module('ingresseSDK')
return $q.reject(error);
}

if (error.code === 3030) {
error.message = 'Não encontramos este e-mail em nossa plataforma. Certifique-se de que ele foi utilizado para criar uma conta na Ingresse.';
Copy link
Contributor

Choose a reason for hiding this comment

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

i think we should remove the Ingresse from this message because this error will show in others whitelables

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@udimberto udimberto mentioned this pull request Jan 8, 2019
@udimberto udimberto added this to the Release 2019.01.a milestone Jan 8, 2019
@udimberto udimberto merged commit 83bd2fc into master Jan 8, 2019
@vitorleal vitorleal deleted the bugfix/password-flow-email branch June 18, 2019 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants