Skip to content

feat(node-menu): bring custom menu items to the node menu. Closes #48, closes #53, closes #25, closes #161#170

Merged
rychkog merged 1 commit intodevelopfrom
custom-menu
Nov 19, 2017
Merged

feat(node-menu): bring custom menu items to the node menu. Closes #48, closes #53, closes #25, closes #161#170
rychkog merged 1 commit intodevelopfrom
custom-menu

Conversation

@rychkog
Copy link
Copy Markdown
Contributor

@rychkog rychkog commented Nov 19, 2017

Extended version of #159 with docs added and some clean ups

@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 19, 2017

Codecov Report

Merging #170 into develop will decrease coverage by 0.01%.
The diff coverage is 89.65%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #170      +/-   ##
===========================================
- Coverage    95.86%   95.84%   -0.02%     
===========================================
  Files           22       22              
  Lines          798      819      +21     
  Branches       117      120       +3     
===========================================
+ Hits           765      785      +20     
- Misses          14       16       +2     
+ Partials        19       18       -1
Impacted Files Coverage Δ
src/tree.types.ts 100% <ø> (ø) ⬆️
src/tree.events.ts 89.18% <100%> (+1.31%) ⬆️
src/tree.service.ts 100% <100%> (+3.17%) ⬆️
src/menu/node-menu.component.ts 100% <100%> (ø) ⬆️
src/menu/menu.events.ts 100% <100%> (ø) ⬆️
src/tree.ts 93.11% <100%> (+0.12%) ⬆️
src/tree-internal.component.ts 93.27% <75%> (-1.51%) ⬇️
src/tree.component.ts 90.19% <75%> (-1.48%) ⬇️

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 68c4dcf...9aacaa5. Read the comment docs.

rychkog pushed a commit that referenced this pull request Nov 19, 2017
@rychkog rychkog merged commit d776886 into develop Nov 19, 2017
@rychkog rychkog deleted the custom-menu branch November 19, 2017 21:28
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.

1 participant