Uploaded image for project: 'Compass '
  1. Compass
  2. COMPASS-4528

Upgrade compass-aggregations to react-ace@9.5.0

    XMLWordPrintable

Details

    • Story
    • Status: Closed
    • Major - P3
    • Resolution: Done
    • None
    • 1.31.0
    • Compass
    • None
    • 3
    • Not Needed
    • Iteration Orlando, Iteration Phnom Penh, Iteration Q

    Description

      Problem Description

      As part of the 3rd Party HIPPA upgrades for mms, we've upgraded to using the newest (9.2.0 9.4.3) version of react-ace. Because @mongodb-js/compass-aggregations has a peer dependency on the old react-ace version, it will also need to be modified to use the latest react-ace version, which does have some breaking changes.

      I've attempted to do the upgrade myself but consistently hit the following (annoyingly minified) error when attempting to render the AggregationsPlugin in mms. If someone with more familiarity with the library could complete and QA this react-ace upgrade, it would unblock our mms upgrade

      index.js:22 Uncaught (in promise) TypeError: Cannot read property 'shape' of undefined
          at Module.<anonymous> (index.js:22)
          at n (index.js:1)
          at index.js:1
          at index.js:1
          at index.js:1
          at Module.<anonymous> (index.js:1)
          at Module.../../../compass-aggregations/lib/index.js (index.js:27)
          at __webpack_require__ (bootstrap:63)
          at Module../project/metrics/components/DataExplorerAggregation.js (index.js:27)
          at __webpack_require__ (bootstrap:63)
      

      Steps to Reproduce

      Expected Results

      Actual Results

      Additional Notes

      Attachments

        Issue Links

          Activity

            People

              leroux.bodenstein@mongodb.com Le Roux Bodenstein
              jack.weir@mongodb.com Jack Weir
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: