Webpack 5 upgrade

[na9da]

Fixes TerriaJS/terriajs#6998
Merge after TerriaJS/terriajs#7351

• Mirrors webpack and SASS changes in Upgrade to webpack 5 terriajs#6998
• Refactors index.js and entry.js
  • Converted modules to use ESM style imports

[pjonsson]

I looked through this and besides the things I commented on, the things that are there looks good to me. Would love to get webpack 5 merged soon so Trivy can stop shouting about critical security vulnerabilities.

[na9da]

Thanks @pjonsson those were good finds, I have fixed them.

[ljowen]

• Do we also need to bump the version in Dockerfile?

[ljowen]

• Looks good! Beside docker comment, just need to add a CHANGES.md line :)

[pjonsson]

• Do we also need to bump the version in Dockerfile?

@ljowen good catch, and there's a second Dockerfile under deploy/docker. And speaking of simple things that get stuck for a long time in the PR queue, #691 fixes a warning in the Dockerfile.

[pjonsson]

@na9da does #720 need to be merged before this?

[ljowen]

• replace now webpack5 is merged

[pjonsson]

@na9da #726 passed the CI before this change (but after the ESLint update), but I just rebased it and it now fails:

ERROR in ./node_modules/terriajs/test/Models/Catalog/CatalogGroups/OpenDataSoftCatalogGroupSpec.ts:6:23
TS2307: Cannot find module 'fetch-mock' or its corresponding type declarations.
    4 | import OpenDataSoftCatalogItem from "../../../../lib/Models/Catalog/CatalogItems/OpenDataSoftCatalogItem";
    5 | import Terria from "../../../../lib/Models/Terria";
  > 6 | import fetchMock from "fetch-mock";
      |                       ^^^^^^^^^^^^
    7 |
    8 | import facets from "../../../../wwwroot/test/ods/facets.json";
    9 | import datasets from "../../../../wwwroot/test/ods/datasets.json";

The difference is from commit 28aa111 to commit 10c2192.

[pjonsson]

@na9da The 0.3.0 release notes state the ~terriajs alias was replaced with relative paths. Is it the release note or this usage that is wrong?

There is also a usage of ~terriajs left in terriajs (TerriaJS/terriajs#7427).

[na9da]

Good spot. I think I got the release notes wrong. It should be ~terriajs-mixins instead.

~terriajs imports style sheets from dependencies in node_modules. Use of tilde for imports is deprecated by sass-loader but still works. I'll remove them in the next release.