Types clash when using along with @ionic/(again again 🙄🙄)

[ionic-team]

Types clash when using along with @ionic/[email protected] (again again 🙄🙄)

[simonhaenisch]

Just found a solution to get around this... you can add a path mapping for @ionic/core to tsconfig.json so that it resolves @ionic/core to the same path for both your project and @ionic/pwa-elements:

{
  "compilerOptions": {
    // ...

    "baseUrl": ".", // needed to make "paths" work
    "paths": {
      "@ionic/core": ["node_modules/@ionic/core"]
    }
  }
}

---

I have a suggestion for a better and cleaner solution though, that would be a bit of a breaking change but solve this long-term: make Ionic an external dependency. That would mean that when including @ionic/pwa-elements as a script, @ionic/core would have to be included first. When using it in a project, @ionic/core should be a peer dependency. Because of how Ionic components are lazy-loaded, I think this approach would work fine. If agreed, I can try and make a PR for that (/cc @mlynch @mhartington)?

[mlynch]

Hey all, we're working on some changes that should permanently fix this. Stay tuned.

[SoftronetBithin]

Just found a solution to get around this... you can add a path mapping for @ionic/core to tsconfig.json so that it resolves @ionic/core to the same path for both your project and @ionic/pwa-elements:

{
  "compilerOptions": {
    // ...

    "baseUrl": ".", // needed to make "paths" work
    "paths": {
      "@ionic/core": ["node_modules/@ionic/core"]
    }
  }
}

I have a suggestion for a better and cleaner solution though, that would be a bit of a breaking change but solve this long-term: make Ionic an external dependency. That would mean that when including @ionic/pwa-elements as a script, @ionic/core would have to be included first. When using it in a project, @ionic/core should be a peer dependency. Because of how Ionic components are lazy-loaded, I think this approach would work fine. If agreed, I can try and make a PR for that (/cc @mlynch @mhartington)?

Thanks @simonhaenisch . It works for me.

[mlynch]

@simonhaenisch we do want to get back to that point, and it should work. Just given everyone's time it wasn't going to work before we hit 1.0 so we needed to punt on that. There's some compiler work still to be done first.

[MCanhisares]

Do we have an update on this?

[jase88]

Workaround for us was to use selective dependency resolutions with yarn

package.json
"resolutions": { "@ionic/core": "4.3.0" },

[corysmc]

Ran into this issue today as well.

[corysmc]

Thanks @simonhaenisch that fixes it for me!

[mlynch]

Does changing this to use peerDependencies resolve it?

[simonhaenisch]

Just saw that you removed Ionic completely... that's another way of making it work 😄 I liked the idea of it being an extension of Ionic (i. e. it would be able to use ion-toast), however there doesn't seem to be a simple way of maintaining that.

[jcesarmobile]

as 1.1.0 doesn't use @ionic/core, this can be closed