-
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
Feature/frontend integration #136
Conversation
web/src/router/index.js
Outdated
@@ -1,12 +1,15 @@ | |||
import {createRouter, createWebHashHistory} from 'vue-router'; | |||
|
|||
import DsProtMain from '../components/dsprot/DsProtMain.vue' |
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.
import DsProtMain from '../components/dsprot/DsProtMain.vue' | |
const DsProtMain = () => import("@/components/dsprot/DsProtMain.vue") |
I think it's nicer to stick to lazy loading routes :)
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.
Seems to work, changed in next commit. What exactly is the benefit here?
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.
I think you forgot to delete the old folder, besides that consider my comments but nothing that must be changed :)
Reason for this PR:
Note that the dsprotection service frontend can currently only run in dev mode, need to configure URL for dsprotection service backend for the production mode.
Checklist:
Select whatever applies: