creativetimofficial / ct-paper-kit-2-pro-angular

Paper Kit 2 PRO Angular version
2 stars 6 forks source link

Navbar and accordion problem with upgraded modules. #6

Closed vardaru closed 5 years ago

vardaru commented 5 years ago

Hi, Hi recently I have to update ct-paper-kit2-pro-angular to angular 7.1.0 and @ng-bootstrap/ng-bootstrap": "4.0.0", since ng build --prod compile problem (we are using paper kit in a production system). ng-bootstrap/NgDatePicker <= 2.2.2 gives an error. And if you update packages to latest ones, one compile issue arises related with JwBootstrapSwitch which is quite easy to remedy by upgrading the imports in components and examples: replace JwBootstrapSwitchModule with JwBootstrapSwitchNg2Module. Then it is ready to compile and most of the application works except: navbar if you try to run in mobile mode show nothing except hamburger icon and accordion components does not expand. Could you please take a look those issues by upgrading and compiling the source codes. Best regards.

vardaru commented 5 years ago

Any news?

chelaruc commented 5 years ago

Hi, @vardaru. Thank you for using our product and sorry for the late response. Can you give me some images with the issue happening?

vardaru commented 5 years ago

Hi Chelaru, Just download paper-kit pro, update the packaces.json with any tool, I use ‘ncu -a’ and run with ‘ng serve’. You will see the problem.

Thank you for support.

iPhone’umdan gönderildi

Chelaru Ciprian notifications@github.com şunları yazdı (11 Ara 2018 16:37):

Hi, @vardaru. Thank you for using our product and sorry for the late response. Can you give me some images with the issue happening?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

chelaruc commented 5 years ago

@vardaru sorry for the late response. Each dependence changes after an update and that's why you got so many errors. Our product's not built to work with all the versions available. That's why you have to keep our dependencies and wait for our updates.

vardaru commented 5 years ago

Thank you for “late" response Ciprian. No I don’t want the product to work with the all versions, I just want it to work with current versions at least. Best regards.

Chelaru Ciprian notifications@github.com şunları yazdı (27 Oca 2019 13:58):

@vardaru https://github.com/vardaru sorry for the late response. Each dependence changes after an update and that's why you got so many errors. Our product's not built to work with all the versions available. That's why you have to keep our dependencies and wait for our updates.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/creativetimofficial/ct-paper-kit-2-pro-angular/issues/6#issuecomment-457907770, or mute the thread https://github.com/notifications/unsubscribe-auth/ABprItSG-ixbEJApfM_6by6OW-B2Q-WJks5vHYY4gaJpZM4YxbpA.

chelaruc commented 5 years ago

@vardaru then why you update the dependencies?

arashbi commented 5 years ago

I have the same problem here. As why update the dependencies, angular 8 is coming up fast, so at least we can go to angular 7 to ease the pain.