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

refactor(tree): support more functions & property #2121

Merged
merged 1 commit into from
Sep 22, 2018

Conversation

simplejason
Copy link
Member

@simplejason simplejason commented Sep 10, 2018

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number:
#1928 #2000 #2062 #1888 #1836 #2168

What is the new behavior?

Does this PR introduce a breaking change?

[x] Yes
[ ] No

API changes (just some important changes, more details can read doc)

before current
[ngModel] [nzData]
[nzDefaultExpandAll] [nzExpandAll]
[nzDefaultExpandedKeys] [nzExpandedKeys]
[nzDefaultCheckedKeys] [nzCheckedKeys]
[nzDefaultSelectedKeys] [nzSelectedKeys]
(nzOnSearchNode) (nzSearchValueChange)

NzTreeFormatEmitEvent

before current
selectedKeys: NzTreeNode[] selectedKeys: string[]
checkedKeys: NzTreeNode[] checkedKeys: string[]
matchededKeys: NzTreeNode[] matchededKeys: string[]
matchededKeys: new matchededKeys: string[]

Other information

@codecov
Copy link

codecov bot commented Sep 10, 2018

Codecov Report

Merging #2121 into master will decrease coverage by 0.05%.
The diff coverage is 94.19%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2121      +/-   ##
==========================================
- Coverage      96%   95.95%   -0.06%     
==========================================
  Files         477      478       +1     
  Lines       11618    11709      +91     
  Branches     1550     1561      +11     
==========================================
+ Hits        11154    11235      +81     
- Misses        132      133       +1     
- Partials      332      341       +9
Impacted Files Coverage Δ
components/tree/nz-tree.module.ts 100% <ø> (ø) ⬆️
components/tree/nz-tree-util.ts 100% <100%> (ø)
components/tree-select/nz-tree-select.component.ts 97.76% <100%> (-0.9%) ⬇️
components/tree/nz-tree-node.ts 91.52% <88.46%> (-2.23%) ⬇️
components/tree/nz-tree.service.ts 93.6% <92.42%> (-2.08%) ⬇️
components/tree/nz-tree-node.component.ts 96.21% <96.06%> (+1.62%) ⬆️
components/tree/nz-tree.component.ts 98.01% <97.18%> (-1.99%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2fde383...a134cb7. Read the comment docs.

@simplejason simplejason changed the title [WIP] refactor(tree): support more functions & property refactor(tree): support more functions & property Sep 12, 2018
@simplejason simplejason changed the title refactor(tree): support more functions & property [WIP] refactor(tree): support more functions & property Sep 18, 2018
@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #2121 into master will decrease coverage by 0.18%.
The diff coverage is 91.4%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2121      +/-   ##
==========================================
- Coverage   96.02%   95.83%   -0.19%     
==========================================
  Files         479      480       +1     
  Lines       11689    11817     +128     
  Branches     1563     1577      +14     
==========================================
+ Hits        11224    11325     +101     
- Misses        131      149      +18     
- Partials      334      343       +9
Impacted Files Coverage Δ
components/tree/nz-tree.module.ts 100% <ø> (ø) ⬆️
components/tree/nz-tree-util.ts 100% <100%> (ø)
components/tree-select/nz-tree-select.component.ts 97.76% <100%> (-0.9%) ⬇️
components/tree/nz-tree.component.ts 85.27% <82.02%> (-14.73%) ⬇️
components/tree/nz-tree-node.ts 91.52% <88.46%> (-2.23%) ⬇️
components/tree/nz-tree.service.ts 93.67% <92.42%> (-2%) ⬇️
components/tree/nz-tree-node.component.ts 96.33% <96.21%> (+1.74%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c70ff04...bfd01a4. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #2121 into master will decrease coverage by 0.18%.
The diff coverage is 91.4%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2121      +/-   ##
==========================================
- Coverage   96.02%   95.83%   -0.19%     
==========================================
  Files         479      480       +1     
  Lines       11689    11817     +128     
  Branches     1563     1577      +14     
==========================================
+ Hits        11224    11325     +101     
- Misses        131      149      +18     
- Partials      334      343       +9
Impacted Files Coverage Δ
components/tree/nz-tree.module.ts 100% <ø> (ø) ⬆️
components/tree/nz-tree-util.ts 100% <100%> (ø)
components/tree-select/nz-tree-select.component.ts 97.76% <100%> (-0.9%) ⬇️
components/tree/nz-tree.component.ts 85.27% <82.02%> (-14.73%) ⬇️
components/tree/nz-tree-node.ts 91.52% <88.46%> (-2.23%) ⬇️
components/tree/nz-tree.service.ts 93.67% <92.42%> (-2%) ⬇️
components/tree/nz-tree-node.component.ts 96.33% <96.21%> (+1.74%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c70ff04...a9acb66. Read the comment docs.

@codecov
Copy link

codecov bot commented Sep 19, 2018

Codecov Report

Merging #2121 into master will decrease coverage by 0.03%.
The diff coverage is 94.73%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2121      +/-   ##
==========================================
- Coverage      96%   95.96%   -0.04%     
==========================================
  Files         481      482       +1     
  Lines       11832    11965     +133     
  Branches     1580     1594      +14     
==========================================
+ Hits        11359    11482     +123     
- Misses        133      134       +1     
- Partials      340      349       +9
Impacted Files Coverage Δ
components/tree/nz-tree.module.ts 100% <ø> (ø) ⬆️
components/tree/nz-tree-util.ts 100% <100%> (ø)
components/tree-select/nz-tree-select.component.ts 97.76% <100%> (-0.9%) ⬇️
components/tree/nz-tree.service.ts 93.67% <92.42%> (-2%) ⬇️
components/tree/nz-tree-node.ts 92.18% <92.85%> (-1.57%) ⬇️
components/tree/nz-tree-node.component.ts 96.29% <96.15%> (+1.7%) ⬆️
components/tree/nz-tree.component.ts 98.47% <97.8%> (-1.53%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3ef8bcf...9a24e96. Read the comment docs.

@simplejason simplejason changed the title [WIP] refactor(tree): support more functions & property refactor(tree): support more functions & property Sep 20, 2018
@simplejason simplejason mentioned this pull request Sep 20, 2018
refactor(tree): support more functions & property

fix(tree): fix demo

fix dir-tree

fix doc

fix doc

fix(tree): support deprecated property

fix(tree): fix interface

fix(tree): fix param

fix(tree): fix param

fix(tree): fix spec

fix(tree): fix spec

fix(tree): fix origin

fix(tree): support get all nodes

fix(tree): fix demo

fix(tree): fix doc

fix(tree): fix doc

fix(tree): fix deprecatedC

fix(tree): fix deprecated

fix(tree): fix deprecated

fix(tree): fix spec

fix(tree): fix console

fix(tree): fix doc

fix(tree): fix doc
hsuanxyz pushed a commit to hsuanxyz/ng-zorro-antd that referenced this pull request Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants