[MI-100] Prevent calls to the API when rendering the grid if unauthenticated.

[joseconsador]

[MI-100]

Save authenticated flag to the database.
Do not display Create Ticket and Dashboard/Grid if the extension is not setup properly.

Steps to reproduce:

1. Login to the magento backend.
2. Go to Zendesk > Configuration.
3. In the General tab, make all fields empty except for Zendesk Domain.
   In the Admin Backend tab, set "Show support tickets on admin dashboard" to "Yes".
4. Click Save.
5. Go to Zendesk > Dashboard

Expected Result:

1. "Couldn't authenticate you" should appear as an error message.
2. On step Support for French, German, Japanese + more #5 - the configuration notice should be displayed

Actual Result:

1. A php warning (attached) appears, along with the standard error message.

1. On step Support for French, German, Japanese + more #5 - a fatal error is thrown and halts the process (attached)

/cc @miogalang @jwswj @mmolina @atroche

References

• Jira link: https://zendesk.atlassian.net/browse/MI-100

Risks

• Medium - Extension functionality may be affected.

QA notes

• Developer mode must be enabled

[miogalang]

Can you put the bumping in a new commit

[miogalang]

👍

[jwswj]

Can't we just check the domain name, username and password/token fields, rather than introducing a new key-value?

[joseconsador]

@jwswj Those values could be incorrect, which will cause the extension to make some API calls that returns unexpected responses.

[jwswj]

I'm on 👎 on this until we find an approach which doesn't involve saving an attribute based on successful auth. The reason being Magento can't always know about auth. What about is the token gets changed from inside Zendesk, this error would still happen. Or if someone changes their subdomain and forgets to update the Magento settings.

[joseconsador]

This line causes an error. 👎