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

Can't login - probably due to mailchimp incorect settings #3612

Closed
pkarw opened this issue Sep 24, 2019 · 6 comments
Closed

Can't login - probably due to mailchimp incorect settings #3612

pkarw opened this issue Sep 24, 2019 · 6 comments
Assignees
Labels
bug Bug reports P2: Important Priority mark - still high ;)
Milestone

Comments

@pkarw
Copy link
Collaborator

pkarw commented Sep 24, 2019

Current behavior

https://www.dropbox.com/s/qn32epafksiv7xc/Screenshot%202019-09-24%2013.02.36.png?dl=0

It's probably because Mailchimp is on the critical path

@pkarw pkarw added the bug Bug reports label Sep 24, 2019
@pkarw pkarw added this to the 1.11.0-rc.1 milestone Sep 24, 2019
@pkarw pkarw added the P2: Important Priority mark - still high ;) label Sep 24, 2019
@andrzejewsky
Copy link
Contributor

@gibkigonzo can you have look at this? I think it's related to https://github.com/DivanteLtd/vue-storefront/blob/develop/core/modules/user/store/actions.ts#L63 because that error is consumed in the template and displays as error notification which you can see on the screenshot (last one), would be nice to find out what where that error comes from.

@andrzejewsky
Copy link
Contributor

@gibkigonzo Unfortunately I couldn't reproduce it for the first sight, but for sure it should occurs when you go to /my-account as not logged user, then you will see login dialog and try to log in.

@gibkigonzo
Copy link
Contributor

gibkigonzo commented Sep 25, 2019

@andrzejewsky I'm working on it. Probably it's caused by wrong api key

{ type: 'http://developer.mailchimp.com/documentation/mailchimp/guides/error-glossary/',
  title: 'API Key Invalid',
  status: 401,
  detail: 'API key has been disabled',
  instance: 'b5686623-b5ef-400f-afeb-01c024bc4d48' }

I will pull this information to the console so that we can check on https://test.storefrontcloud.io

@pkarw
Copy link
Collaborator Author

pkarw commented Sep 25, 2019

It's surely caused by wrong api key. But ppl. might have wrong keys. It should display the error message probably but don't block the login procedure. It shouldn't be on critical path

@pkarw
Copy link
Collaborator Author

pkarw commented Sep 25, 2019

I'd say that even we shouldn't show the error message when reading the subscription status; only when subscribing/desubscribing as it was before in 1.10/1.9 - can you please fix it to the previous state?

@pkarw
Copy link
Collaborator Author

pkarw commented Sep 25, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports P2: Important Priority mark - still high ;)
Projects
None yet
Development

No branches or pull requests

3 participants