Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Changed detectChanges() to markForCheck() to avoid ViewDestroyedError #21

Merged
merged 2 commits into from
Apr 23, 2019

Conversation

Blackbaud-AlexKingman
Copy link
Contributor

@Blackbaud-AlexKingman Blackbaud-AlexKingman commented Apr 23, 2019

@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #21   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          19     19           
  Lines         647    647           
  Branches       96     96           
=====================================
  Hits          647    647
Impacted Files Coverage Δ
src/app/public/modules/tabs/tabset.component.ts 100% <100%> (ø) ⬆️

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 e93b456...26596ee. Read the comment docs.

Copy link
Member

@Blackbaud-TrevorBurch Blackbaud-TrevorBurch left a comment

Choose a reason for hiding this comment

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

LGTM

@Blackbaud-AlexKingman Blackbaud-AlexKingman merged commit 71a61fe into master Apr 23, 2019
@Blackbaud-AlexKingman Blackbaud-AlexKingman deleted the ViewDestroyedError-fix branch April 23, 2019 20:40
@Blackbaud-AlexKingman Blackbaud-AlexKingman changed the title Removed setTimeout to avoid ViewDestroyedError Changed detectChanges() to markForCheck() to avoid ViewDestroyedError Apr 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants