Skip to content

Remove stale webpack config that injects superGet.#673

Merged
tephenavies merged 1 commit into
mainfrom
remove-superget-subst
May 2, 2024
Merged

Remove stale webpack config that injects superGet.#673
tephenavies merged 1 commit into
mainfrom
remove-superget-subst

Conversation

@na9da

@na9da na9da commented May 2, 2024

Copy link
Copy Markdown
Contributor

This was used with mobx4 to replace babel's getter with a mobx aware version. With mobx6, we no longer require this.

This was used with mobx4 to replace babel's getter with a mobx aware
version. With mobx6, we no longer require this.
@tephenavies tephenavies merged commit 911e952 into main May 2, 2024
@tephenavies tephenavies deleted the remove-superget-subst branch May 2, 2024 05:40
@pjonsson

pjonsson commented May 7, 2024

Copy link
Copy Markdown
Contributor

@na9da with this merged, should string-replace-loader be removed from package.json in terriajs?

@na9da

na9da commented May 7, 2024

Copy link
Copy Markdown
Contributor Author

@pjonsson - yes. I had a PR for that, but it needs a bit of rework to adjust the credits display. If there's separate PR to just remove string-replace-loader, I can merge that one quickly.

@pjonsson

pjonsson commented May 8, 2024

Copy link
Copy Markdown
Contributor

@na9da ok, TerriaJS/terriajs#7145 just removes the dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants