[ABTEST-438] Handle and display ABn Testing error in UI Created: 13/Apr/21  Updated: 24/May/21  Resolved: 17/May/21

Status: Closed
Project: A/B Testing
Component/s: None
Affects Version/s: None
Fix Version/s: 1.0

Type: Story Priority: Neutral
Reporter: Nguyen Phung Chi Assignee: Chuong Doan Huy
Resolution: Fixed Votes: 0
Labels: VN-Analysis, VN-Implementation, VN-Testing
Remaining Estimate: Not Specified
Time Spent: 3d 3.25h
Original Estimate: Not Specified

Attachments: PNG File Screenshot 2021-05-18 at 09.48.13.png     PNG File abn-app-error.png     PNG File image-2021-04-13-15-29-24-915.png     PNG File page-app-error.png    
Issue Links:
Cloners
is cloned by ABTEST-456 Invalid license. Handle and display A... Closed
Template:
Acceptance criteria:
[ ]* Error should not block the functionality of Pages app
[ ]  Show friendly error message on each row (timeout)
[ ]  Log warning message in case no apigateway config yet
Task DoD:
[X]* Doc/release notes changes? Comment present?
[X]* Downstream builds green?
[X]* Solution information and context easily available?
[X]* Tests
[X]* FixVersion filled and not yet released
[X]  Architecture Decision Record (ADR)
Documentation update required:
Yes
Date of First Response:
Epic Link: ABn GA tasks
Sprint: Content Mngmt 03, Content Mngmt 04
Story Points: 5

 Description   

Situation

In ABn Testing app, we are throwing the exception to UI, for instance invalid license, unauthorized API gateway when getting the test results, they made the app broken and can't use anymore.

Same situation happened in Pages app, because there is a decoration column "AB testing status"

 

Approach/ Proposal

  • The error should not break the app
  • Currently, it will displayed multiple times due to exception thrown for each AB test. So a notification should displayed one time

 

Scenarios

  • Use case --> Invalid apiGateWayURL, timeout connection or NullPointerException...
  • Use case 2 --> The license becomes invalid after 1-year (for example) if not re-newed  

AC

  • Exception should not throw on the screen
  • General error message like "Unable to retrieve test results" displayed on each row if exception happens (see attachment abn-app-error and page-app-error)


 Comments   
Comment by Chuong Doan Huy [ 12/May/21 ]

First proposal is to display a notification on abtesting app and page app whenever error happened.

But because each row (test) may have different status/errors, showing notification multiple times at once make users feel annoying/uncontrollable/overwhelmed.

We can customize abtesting app to show general error message one time, but this is not specific to any row (test). Other than that, the blocking issue for us is : we can customize abtesting app, but can not customize pages app to show error the way we want as this will break pages app behaviors.

The possible recommended solution (after discuss with nguyen.phung and oanh.thai ) is to display error message in each row (detail error will be logged, admin can trace easily), in the test state column (see attached images). This way, each row can have different errors and is easy to address.

Generated at Sun Feb 11 22:56:14 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.