FLUID-5355: Climate Change demo should handle the case where the TTS server is unavailable.

Metadata

Source
FLUID-5355
Type
Bug
Priority
Critical
Status
Closed
Resolution
Won't Fix
Assignee
N/A
Reporter
Justin Obara
Created
2014-05-01T11:45:36.157-0400
Updated
2015-06-26T09:57:42.400-0400
Versions
  1. 1.5
Fixed Versions
  1. 1.9
Component
  1. Demos
  2. Prefs Framework

Description

The climate change demo's text-to-speech feature requires access to a server hosted at the IDRC. The demo should ensure to handle the case where the server is down or otherwise inaccessible (e.g. running locally without internet access).

Comments

  • Justin Obara commented 2014-05-01T11:46:01.777-0400

    A first step solution may be to just use a browser alert to notify the user that the server is unavailable.

  • Justin Obara commented 2014-05-01T13:32:16.424-0400

    @@Michelle D'Souza I attempted to implement a simple solution for this using the onError event from the SelfVoicingEnactor:
    https://github.com/fluid-project/infusion/blob/master/src/demos/prefsFramework/js/SelfVoicingEnactor.js#L103-L106

    This works okay in Firefox, but the event isn't triggered in Chrome. It appears that Chrome throws an error as soon as the resource is inaccessible, preventing our error event pipeline from triggering.

    Is it enough to have this working just in Firefox? (only firefox and chrome support the TTS enactor).

  • Justin Obara commented 2014-05-02T10:30:28.127-0400

    I did some more experimentation with this to see if I could use a try-catch block to get at the error, but this didn't work either. This is possibly a result of asynchrony in retrieving the resource.

    It appears that the library we are using to manage the audio, has an open issue for this https://github.com/jaysalvat/buzz/issues/64

  • Justin Obara commented 2014-05-05T12:22:19.073-0400

    Submitted a pull request to handle the error in Firefox.
    https://github.com/fluid-project/infusion/pull/511

  • Cindy Li commented 2014-05-05T15:10:20.985-0400

    The pull request has been merged into the project repo @ 2a94c0771e675fabf51f617c8d958e87d29e4f4f

    Note that the fix pops up the alert window in firefox but not in chrome.

  • Justin Obara commented 2015-06-15T14:45:42.661-0400

    The TTS Server is no longer being used. Instead, TTS is supplied via the web speech api. Browsers that don't support the web speech api, do not show the adjuster.

  • Justin Obara commented 2015-06-26T09:50:49.805-0400

    Reopening to change the fix version to 1.9