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

optimize: saga transaction editor enhancements #2464

Merged
merged 6 commits into from
Apr 16, 2020

Conversation

mxszs
Copy link
Contributor

@mxszs mxszs commented Mar 26, 2020

Ⅰ. Describe what this PR did

  1. The default line is the fillet line

  2. Compensation type post connection point cancel arrow

  3. Node border of substatemachine type is bold

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@mxszs mxszs changed the title feat(sagadesigner):1. 默认连线为圆角折线 2.Compensation类型后置连接点取消箭头 3. SubState… feat(sagadesigner):Saga transaction editor enhancements Mar 26, 2020
@slievrly slievrly requested a review from long187 March 27, 2020 03:57
@@ -35,6 +35,15 @@ class Page extends React.Component {
}

if (newData !== oldData) {
// Compensation类型后置连接点去掉箭头
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use English comment instead.

@@ -38,12 +38,24 @@ class WorkSpaceBase extends React.Component {
const { executeCommand, update } = propsAPI;

if (param.action == 'add' && param.item.type == 'edge') {
// 默认圆角折线
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use English comment instead.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok

@codecov-io
Copy link

codecov-io commented Mar 31, 2020

Codecov Report

Merging #2464 into develop will decrease coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2464      +/-   ##
=============================================
- Coverage      51.51%   51.43%   -0.08%     
+ Complexity      2667     2662       -5     
=============================================
  Files            523      529       +6     
  Lines          16944    16956      +12     
  Branches        2053     2051       -2     
=============================================
- Hits            8728     8722       -6     
- Misses          7390     7408      +18     
  Partials         826      826              
Impacted Files Coverage Δ Complexity Δ
...rce/sql/struct/cache/PostgresqlTableMetaCache.java 76.08% <0.00%> (-3.46%) 10.00% <0.00%> (ø%)
...o/seata/server/coordinator/DefaultCoordinator.java 54.08% <0.00%> (-0.52%) 28.00% <0.00%> (-1.00%)
...in/java/io/seata/server/session/GlobalSession.java 85.32% <0.00%> (-0.46%) 71.00% <0.00%> (-1.00%)
...a/server/storage/db/lock/LockStoreDataBaseDAO.java 56.99% <0.00%> (-0.23%) 26.00% <0.00%> (ø%)
...io/seata/server/storage/db/lock/LockStoreSqls.java
...eata/core/store/db/sql/lock/MysqlLockStoreSql.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
.../core/store/db/sql/lock/OceanbaseLockStoreSql.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...a/core/store/db/sql/lock/AbstractLockStoreSql.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...core/store/db/sql/lock/PostgresqlLockStoreSql.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
...ata/core/store/db/sql/lock/OracleLockStoreSql.java 0.00% <0.00%> (ø) 0.00% <0.00%> (?%)
... and 2 more

@l81893521 l81893521 added the first-time contributor first-time contributor label Apr 9, 2020
@mxszs mxszs requested a review from l81893521 April 15, 2020 10:31
Copy link
Contributor

@long187 long187 left a comment

Choose a reason for hiding this comment

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

I have downloaded and passed the test

@slievrly slievrly changed the title feat(sagadesigner):Saga transaction editor enhancements optimize: saga transaction editor enhancements Apr 16, 2020
Copy link
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

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

LGTM

@slievrly slievrly added this to the 1.2.0 milestone Apr 16, 2020
@slievrly slievrly merged commit 4d0a1c7 into apache:develop Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
first-time contributor first-time contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants