Skip to content
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

EventMesh Dashboard: Configuration #1046

Merged
merged 3 commits into from
Aug 2, 2022
Merged

Conversation

xiaoyang-sde
Copy link
Member

@xiaoyang-sde xiaoyang-sde commented Jul 27, 2022

Motivation

The dashboard page displays the current configuration of EventMesh.

Screen Shot 2022-07-27 at 12 17 51 PM

Screen Shot 2022-07-30 at 5 07 43 PM

Xiaoyang Liu added 2 commits July 26, 2022 22:58
@xiaoyang-sde xiaoyang-sde self-assigned this Jul 27, 2022
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has totally checked 1123 files.

Valid Invalid Ignored Fixed
809 2 312 0
Click to see the invalid file list
  • eventmesh-dashboard/next-env.d.ts
  • eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/admin/response/GetConfigurationResponse.java

@codecov
Copy link

codecov bot commented Jul 27, 2022

Codecov Report

Merging #1046 (db284a2) into dashboard (f66bc64) will increase coverage by 0.35%.
The diff coverage is 0.99%.

@@              Coverage Diff               @@
##             dashboard   #1046      +/-   ##
==============================================
+ Coverage         8.94%   9.30%   +0.35%     
- Complexity         533     587      +54     
==============================================
  Files              349     361      +12     
  Lines            21797   22944    +1147     
  Branches          2426    2516      +90     
==============================================
+ Hits              1950    2134     +184     
- Misses           19696   20617     +921     
- Partials           151     193      +42     
Impacted Files Coverage Δ
...e/eventmesh/common/config/CommonConfiguration.java 80.95% <0.00%> (-6.71%) ⬇️
...he/eventmesh/common/protocol/SubscriptionItem.java 0.00% <ø> (ø)
...esh/common/protocol/SubscriptionModeConverter.java 0.00% <0.00%> (ø)
...esh/common/protocol/SubscriptionTypeConverter.java 0.00% <0.00%> (ø)
...entmesh/common/protocol/http/HttpEventWrapper.java 0.00% <0.00%> (ø)
...esh/common/protocol/http/common/ClientRetCode.java 0.00% <0.00%> (ø)
...tmesh/common/protocol/http/common/ProtocolKey.java 0.00% <0.00%> (ø)
...ntmesh/common/protocol/http/common/RequestURI.java 0.00% <0.00%> (ø)
...che/eventmesh/common/protocol/tcp/codec/Codec.java 62.22% <ø> (-1.22%) ⬇️
...entmesh/common/utils/ConfigurationContextUtil.java 0.00% <0.00%> (ø)
... and 96 more

Help us with your feedback. Take ten seconds to tell us how you rate us.

@Alonexc
Copy link
Contributor

Alonexc commented Jul 28, 2022

The license check did not pass, please check if the license was added, or the format is not accurate?

@xiaoyang-sde
Copy link
Member Author

The license check did not pass, please check if the license was added, or the format is not accurate?

Yes I will add the missing licenses later.

@xiaoyang-sde xiaoyang-sde changed the base branch from master to dashboard July 30, 2022 23:31
@xiaoyang-sde xiaoyang-sde changed the title [WIP] EventMesh Dashboard: Configuration EventMesh Dashboard: Configuration Jul 30, 2022
@xiaoyang-sde xiaoyang-sde marked this pull request as ready for review July 31, 2022 00:00
@xiaoyang-sde xiaoyang-sde force-pushed the dashboard-configuration branch from 6207a6b to db284a2 Compare July 31, 2022 00:07
@xwm1992 xwm1992 merged commit 9f7283f into dashboard Aug 2, 2022
@xiaoyang-sde xiaoyang-sde deleted the dashboard-configuration branch August 2, 2022 03:31
xwm1992 added a commit to xwm1992/EventMesh that referenced this pull request Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants