Chrome-Store-Foxified icon indicating copy to clipboard operation
Chrome-Store-Foxified copied to clipboard

If AMO credentials fail, educate user

Open Madis0 opened this issue 6 years ago • 6 comments

If the extension is unable to connect to AMO in order to sign the extension, it should instruct the user:

  • why it needs to connect to AMO
  • how to sign up and log in
  • offer the unsigned version of the extension as a link (+ the popup tutorial on how to install if needed)

Madis0 avatar Nov 10 '17 20:11 Madis0

It should currently already offer "Install Unsigned":

Is this not showing up?

I need to definitely add a popup/tutorial though.

Noitidart avatar Nov 11 '17 00:11 Noitidart

No, the dashboard has both "install unsigned" and a tutorial. I was talking about this pointless page:

Madis0 avatar Nov 11 '17 07:11 Madis0

Oh!!! That's real bad it should never happen. What this means is that it was not able to find the "user key" and "user secret" from this page: https://addons.mozilla.org/en-US/developers/addon/api/key/

Are you able to reproduce this? I haven't been able to get this error to show up.

Noitidart avatar Nov 11 '17 14:11 Noitidart

Yes, I have got that for every extension. The reason is probably that I browse Chrome Web Store in a defined Container but AMO in a generic Container, haven't bothered to change that 😁

Madis0 avatar Nov 11 '17 14:11 Madis0

Oh way interesting I'll try that out, for sure if AMO is in a generic container, it is not able to get those cookies, but it should give error "not logged in". I'll try to reproduce that, thanks very much for your fast reply!

Noitidart avatar Nov 11 '17 14:11 Noitidart

This issue is not the same as but related to - https://github.com/Noitidart/Chrome-Store-Foxified/issues/89

For #89 I have a fix coming up for 3.3 - but in case it still fails (it should never fail) we can create a dialog. But because it should never ever fail we can put this to low priority.

However it would be maybe nice to stick in somewhere why it needs to connect to AMO. Definitely open to ideas on where to put that, and definitely open to contributions adding this in! :)

Noitidart avatar Nov 12 '17 14:11 Noitidart