Skip to content

Conversation

@joseconsador
Copy link
Contributor

Do not display Create Ticket and Dashboard/Grid if the extension is not setup properly.

related PR: #77

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.

screen shot 2015-06-22 at 3 48 57 pm

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

screen shot 2015-06-22 at 4 00 41 pm

/cc @miogalang @jwswj @mmolina @iandjx

References

QA notes

  • Developer mode must be enabled

Risks

  • Medium - Extension functionality may be affected.

@miogalang
Copy link
Contributor

👍

1 similar comment
@jwswj
Copy link
Contributor

jwswj commented Sep 15, 2015

👍

joseconsador added a commit that referenced this pull request Sep 16, 2015
[MI-100] Check connection status before loading the grid
@joseconsador joseconsador merged commit 9978fea into master Sep 16, 2015
@joseconsador joseconsador deleted the jose/MI-100-connection-status branch September 16, 2015 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants