avatar PhilETaylor
PhilETaylor
9 Jun 2020

Steps to reproduce the issue

Patch Tester 4.0.0 Release Candidate 2
Install a patch
Click fetch

Expected result

Error message to tell me something

Actual result

Nothing appears to happen - left at this screen:

Screenshot 2020-06-09 at 21 09 23

the Ajax response contains an error not displayed to the user

{"success":false,"message":"Cannot fetch data from GitHub while there are applied patches. Please revert those patches before continuing.","messages":null,"data":null}
avatar PhilETaylor PhilETaylor - open - 9 Jun 2020
avatar PhilETaylor PhilETaylor - change - 9 Jun 2020
The description was changed
avatar PhilETaylor PhilETaylor - edited - 9 Jun 2020
avatar PhilETaylor PhilETaylor - change - 9 Jun 2020
The description was changed
avatar PhilETaylor PhilETaylor - edited - 9 Jun 2020
avatar particthistle
particthistle - comment - 18 Jul 2020

Based on the workaround that's been provided to me, the fetch data process could avoid this error occurring by checking the number of records in #_patchtester_tests first. It would then trigger an alert if the record count is >0, as that's where the data is stored for the applied patches.

If you wanted to improve the UX there you could look at:

  • display an alert message "Please revert patches that are currently applied before fetching fresh Github Data"
  • filters the patch list to the list of applied patches so the user can revert patches immediately.
avatar roland-d
roland-d - comment - 25 Jul 2020

This has been fixed in the upcoming release, to show the message in the modal.

avatar PhilETaylor PhilETaylor - change - 25 Jul 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-07-25 13:17:37
Closed_By PhilETaylor
avatar PhilETaylor PhilETaylor - close - 25 Jul 2020

Add a Comment

Login with GitHub to post a comment