creativetimofficial / ct-impact-design-system-pro

Impact Design System PRO
https://demos.creative-tim.com/impact-design-system-pro/
3 stars 2 forks source link

[Bug] 3rd party software included in download #6

Closed smaddock closed 4 years ago

smaddock commented 4 years ago

Version

1.0.0

Reproduction link

https://demos.creative-tim.com/impact-design-system-pro/

Operating System

n/a

Device

n/a

Browser & Version

n/a

Steps to reproduce

n/a

What is expected?

All dependencies are managed through NPM, including Bootstrap, to keep repos as lean as possible.

What is actually happening?

Just looking at the Dashboard directory: /src/dashboard/assets/vendor/ /src/dashboard/assets/js/vendor/ /src/dashboard/assets/js/components/vendor/ /src/dashboard/scss/bootstrap/ all contain third-party software directly, outside of /node_modules/ directory that downloads when running NPM.


Solution

Update all references, normalize all of the dependencies, and bring them exclusively through NPM.

Additional comments

Zoltán at Themesburg said an update addressing this would be ready by next week; submitting here as well for completeness.

zoltanszogyenyi commented 4 years ago

Thanks @smaddock for pointing this out. As discussed in the email, I believe all third party dependencies from the dashboard folder as well will be kept through the NPM manager.

zoltanszogyenyi commented 4 years ago

@smaddock Tomorrow the update will be live.

zoltanszogyenyi commented 4 years ago

Update: due to a complication with the libraries the update will be postponed for tomorrow.

zoltanszogyenyi commented 4 years ago

Except for about 4 libraries, all of them have been transferred to NPM in version 1.1.