search

ionic-team / ionic-pwa-toolkit

Build lightning fast Progressive Web Apps with zero config and best practices built-in. Go from zero to production ready with Ionic and Stencil (Web Components).
MIT License
633 stars 74 forks source link

@ionic/core/... Type 'keyof B' does not satisfy the constraint 'string'. Type 'string | number | symbol' is not assignable to type 'string'. Type 'number' is not assignable to type 'string'. #70

Closed peterennis closed 6 years ago

peterennis commented 6 years ago

Stencil version: (run npm list @stencil/core from a terminal/cmd prompt and paste output below):

@stencil/core v0.9.7

I'm submitting a ... (check one with "x") [x] bug report [ ] feature request [ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or https://stencil-worldwide.slack.com

Current behavior: Build fail

capture321

Expected behavior: Starter works

Steps to reproduce:

Other information:

C:\ae\adaept.com\aepwa>ionic info
[WARN] You are not in an Ionic project directory. Project context may be missing.
cli packages: (C:\Users\peter\AppData\Roaming\npm\node_modules)

   @ionic/cli-utils  : 2.0.0-rc.6
   ionic (Ionic CLI) : 4.0.0-rc.6

System:

   NodeJS : v8.11.1
   npm    : 6.1.0-next.0
   OS     : Windows 10

C:\ae\adaept.com\aepwa>

I ran npm audit fix and it reported corrected 4 security errors.

Repo: https://github.com/peterennis/aepwa

adamlacombe commented 6 years ago

Duplicate of #68

Fix: @ionic/core#15dc651

peterennis commented 6 years ago

Not fixed in @ionic/core@4.0.0-alpha.7 - maybe in the next release.

capture015

pedrotcm commented 6 years ago

Not fixed in @ionic/core@4.0.0-alpha.7.

adamlacombe commented 6 years ago

Yes - the upcoming release has the fix. For the time being you can replace <keyof B> with <any> in node_modules/@ionic/core/dist/types/utils/overlays.d.ts

adamdbradley commented 6 years ago

This is fixed as of alpha.10. Thanks