-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rebuild web client using npm, ts and vuejs #1
Conversation
TODO need to change for production
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall noice 1. I could not really go deep. I have no clue what excalty your doing. But the weir indent make it hard to read. And some files are javascript. I am not sure why. But still goood job.
web/client/src/components/infrastructure/infrastructure-map.vue
Outdated
Show resolved
Hide resolved
web/client/src/components/golden-layout-components/SimulationComponent.vue
Outdated
Show resolved
Hide resolved
web/client/src/components/infrastructure/infrastructure-map.vue
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
N1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job
# Conflicts: # .github/workflows/linux.yml # cmake/soro-server-client.cmake # web/client/index.html # web/client/utl/goldenLayoutHelper.js
This PR refactors the web client to use TypeScript, Vue.js, VueX and NPM. It thus enhances the development and production deployment experience for the webclient as before just about plain javascript was used.
Having installed npm, the setup and starting the HMR dev-server is rather easy:
Production deployment gets a whole lot easier too:
The bulk load of this PR is the
package-lock.json
(abt 5000 lines) and the removal of the hardcoded dependencies. The following packages got upgraded with this PR:This PR also disables clearing the cache, as it was not usable before the PR too. Trying the clear the cache resulted in an error, so it was disabled:
Closes #4