DECA-249: Errors returned by the server are not handled by the UI

Metadata

Source
DECA-249
Type
Bug
Priority
Critical
Status
Closed
Resolution
Fixed
Assignee
N/A
Reporter
Justin Obara
Created
2012-05-14T14:08:18.197-0400
Updated
2012-12-03T11:05:10.801-0500
Versions
  1. 0.5
Fixed Versions
  1. 0.7
Component
  1. Export

Description

If an error occurs on import and server, an error message is returned to the client. However, the client doesn't do anything with it.

Steps to reproduce:

1) Launch Decapod

2) Attempt to import and generate a pdf from non-image files
Notice that the progress spinner appears to continue forever; however if you look in the server logs and/or firebug you will see that a 415 error was returned.

Environments

FF 12 (Ubuntu 10.04)

Attachments

Comments

  • Jonathan Hung commented 2012-09-07T12:46:32.563-0400

    We need to get a complete list of errors so we can create a UI to handle them.

    • Error should be reported to UI
  • Justin Obara commented 2012-11-05T12:36:53.802-0500

    possible errors:

    1) image conversion or pdf generation fail (status set to "error")
    2) import is not an image (return 415 error)
    3) busy generating an export (return 409 error)

  • Jonathan Hung commented 2012-11-06T16:50:21.101-0500

    In the case where the user uploads a single file and it is not a valid image file, error messages are split up across the UI. I think these messages should appear in the same place - namely the Export section and not in the Uploader.

    See attached image.

    Also, in the case of a genpdf error, there should be Help and Restart link displayed in the Export option.

  • Justin Obara commented 2012-11-07T08:47:09.568-0500

    These are actually 2 different errors. The first one is that the server rejected the file for not being an image. The second error, the one in the export, is because it attempted to create an export with no images.

    As for the links we should be able to add those in. If I'm not mistaken, the description blocks can accept markup.