Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-12619

Don't use ECMAScript6 in Admin UI

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Won't Fix
    • None
    • None
    • Admin UI
    • None

    Description

      Spinoff from SOLR-8207

      Since we don't traspile our JS but send it straight to endusers browsers, we should make sure to stay on language level 5 of JS/ES.

      In stream.js, the keyword let is used which is an ES6 feature. Dennis Gove will you have a look? This is code from 2006 so probably most users have modern browsers, but still..

      We should also look through rest of JS for similar warnings.

      Attachments

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            janhoy Jan Høydahl
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment