-
Notifications
You must be signed in to change notification settings - Fork 76
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/UI/framework/vue3 #111
Conversation
`Vue3` `Vite` `TypeScript` `Ant Design Vue` `AntV|echarts` `lodash` `i18n` `colorful theme`
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.
LGTM
) | ||
|
||
response.use( | ||
(response) => { |
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.
Wouldn't it be better to add some global exception handling hints?
Do we need to add login verification in the future?
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.
please create a issue about the global exception and login
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #111 +/- ##
==========================================
- Coverage 48.53% 48.52% -0.01%
==========================================
Files 149 148 -1
Lines 9565 9568 +3
Branches 41 41
==========================================
+ Hits 4642 4643 +1
- Misses 4395 4397 +2
Partials 528 528 ☔ View full report in Codecov by Sentry. |
…b/dubbo-kubernetes into feature/ui/framework/vue3 * 'feature/ui/framework/vue3' of https://github.com/Helltab/dubbo-kubernetes: support grafana configuration (apache#108)
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.
LGTM.
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.
LGTM.
fix these issues #76 #106 #104
Vue3
Vite
TypeScript
Ant Design Vue
AntV|echarts
lodash
i18n
colorful theme