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

feat: remove fields scheduled to be removed in 4.0 #1439

Merged
merged 1 commit into from
Apr 29, 2019
Merged

feat: remove fields scheduled to be removed in 4.0 #1439

merged 1 commit into from
Apr 29, 2019

Conversation

yggg
Copy link
Contributor

@yggg yggg commented Apr 29, 2019

Please read and mark the following check list before creating a pull request:

Short description of what this resolves:

BREAKING CHANGE:
Popover:
'nbPopoverMode' input removed. Use 'nbPopoverTrigger' instead.
NbPopoverDirective 'mode' property removed. Use 'trigger' instead.

NbSelectComponent 'selectionChange' removed. Use nb-select (selected)
binding to track selection change and <nb-option (click)=""> to track
option click.
@codecov
Copy link

codecov bot commented Apr 29, 2019

Codecov Report

Merging #1439 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1439      +/-   ##
==========================================
+ Coverage   82.57%   82.59%   +0.02%     
==========================================
  Files         240      240              
  Lines        7327     7320       -7     
  Branches      656      656              
==========================================
- Hits         6050     6046       -4     
+ Misses       1088     1085       -3     
  Partials      189      189
Impacted Files Coverage Δ
...work/theme/components/popover/popover.directive.ts 100% <ø> (+8.82%) ⬆️
...mework/theme/components/select/select.component.ts 87.1% <100%> (-0.15%) ⬇️

@yggg yggg merged commit a03c53f into akveo:master Apr 29, 2019
yggg added a commit that referenced this pull request May 27, 2019
BREAKING CHANGE:
Popover:
'nbPopoverMode' input removed. Use 'nbPopoverTrigger' instead.
NbPopoverDirective 'mode' property removed. Use 'trigger' instead.

NbSelectComponent 'selectionChange' removed. Use nb-select (selected)
binding to track selection change and <nb-option (click)=""> to track
option click.
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.

None yet

1 participant