Why is sf a package suggestion for modules?

[tmelliott]

This will cause some issues as it uses requireNamespace('sf') without being inside an if.

Ideally, anything requiring sf should instead access functions inside iNZightMaps which can do the bulk of the work.

This is probably a significant amount of work, so I wont adjust it now (and instead manually remove sf from the build).

[daniel-barnett]

This is to allow the whole preview thing while choosing maps (I think...). I should be able to refactor that into a function in maps to eliminate the dependency

[tmelliott]

That would be great. Ideally the Modules GUI just calls functions inside the other package, which is especially useful for replicating on Lite.