kytos-ng / ui

Kytos Admin UI
MIT License
0 stars 3 forks source link

Bundle napps with webpack #88

Open HeriLFIU opened 2 months ago

HeriLFIU commented 2 months ago

Issue

Napps are currently being loaded in at runtime and can't be tested properly, so they need to be bundled in with current/main UI.

Current Fix

Steps:

italovalcy commented 1 month ago

Hi @HeriLFIU, even if you bundle the Napps together with the UI, since all the Napps strongly depend on Kytos REST APIs, wouldn't that impact the tests anyway? I'm not sure how you will benefit from having the Napps bundled with the UI core but not having all APIs available. I apologize in advance if I missed something, but it may be something to keep in mind.

HeriLFIU commented 1 month ago

Hello @italovalcy, Yes, that is true. I had originally thought that the only way to test the UI with the Vue compatibility layer was in development mode with the dev server. I had also thought that it would be the fastest way of testing since I could just save and view the changes. But, after working with Webpack today, I realized that I could enable it on production as well and get the same results. Some of the code written is not yet lost, as it can probably be used within the other async task to bundle the napps individually and asynchronously load them at runtime. I was also able to read the vue compat documentation, so I should be able to change some of the code much quicker.