ICEpush
  1. ICEpush
  2. PUSH-289

Include extension scripts when available

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Bridge-it 1.0
    • Fix Version/s: Bridge-it 1.0
    • Component/s: Push Server
    • Labels:
      None
    • Environment:
      ICEpush, NaaS
    • Assignee Priority:
      P1
    • Affects:
      Compatibility/Configuration

      Description

      Currently ICEpush makes use of two JavaScript files, served via code.icepush and mobi.icepush. ICEfaces contained detection code that would include mobi.icepush automatically, but this is not available to NaaS users.

      The new default behavior should be to concatenate and serve the complete available JavaScript for code.icepush. Since this is a change in default behavior, the configuration parameter org.icepush.serveExtensionScripts can be set to false to obtain the original behavior.

        Activity

        Hide
        Ted Goddard added a comment -

        There is likely no harm in including mobi.icepush twice so ICEfaces/ICEmobile may not require any immediate change.

        Show
        Ted Goddard added a comment - There is likely no harm in including mobi.icepush twice so ICEfaces/ICEmobile may not require any immediate change.
        Hide
        Ted Goddard added a comment -

        The window.onload listener probably shouldn't have two copies – mobi.icepush should be modified so that including it multiple times doesn't cause harm.

        Show
        Ted Goddard added a comment - The window.onload listener probably shouldn't have two copies – mobi.icepush should be modified so that including it multiple times doesn't cause harm.
        Hide
        Mircea Toma added a comment -

        Modified EE MainServlet to serve its additional JS code along with the ICEpush base code.

        Show
        Mircea Toma added a comment - Modified EE MainServlet to serve its additional JS code along with the ICEpush base code.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: