SAFE-template icon indicating copy to clipboard operation
SAFE-template copied to clipboard

Safe apps have npm dependencies missing / out of date

Open teknikal-wizard opened this issue 4 years ago • 5 comments

I just installed a package with Femto, and it bumped my React version to 17.x (from 16.x in the default app) and also imported bulma.

It looks like bulma is being pulled in because Fulma is still in the paket dependencies file. We could just remove it now that we use Feliz.Bulma (which doesn't require bulma at all).

React is being bumped because of Feliz I believe.

teknikal-wizard avatar Jul 15 '21 16:07 teknikal-wizard

Feliz.Bulma has bulma added as a requirement for Femto.

Nhowka avatar Jul 16 '21 01:07 Nhowka

Ah so it does. It's a bit confusing as the template doesn't come with Bulma but it still seems to work?

Either way, I would think it is a good idea to release a template update after running Femto on it.

teknikal-wizard avatar Jul 16 '21 08:07 teknikal-wizard

The template kinda cheats and gets it from a CDN, that's why it don't appear to be a dependency. Check here

Nhowka avatar Jul 16 '21 12:07 Nhowka

ahhhh

teknikal-wizard avatar Jul 16 '21 16:07 teknikal-wizard

What's the best way forward here - should we be including it again as a dependency? A couple of of the reasons we removed it was:

  1. Cut down on bundle size.
  2. Ensure we kept the webpack stuff as simple as possible.

isaacabraham avatar Jul 17 '21 21:07 isaacabraham

@isaacabraham just doing the clean up - I'll close this for now as the current template uses React 17. The Bulma npm vs CDN issue can be reopened if there is interested.

teknikal-wizard avatar Nov 11 '22 15:11 teknikal-wizard