Closed kylecannon closed 3 years ago
Thanks for the feedback @brandonroberts. Feedback is addressed.
@kylecannon will you rebase on master? Had to move some things around with the name change
@brandonroberts done. Still trying to see what needs to be done though to get this merged. I commented above with a couple of suggestions.
async pipe typically emits a null before the value which causes everything to blow up. To correct this, I did the following:
linkTo
andlinkActive
.There may be other instances I am missing, but this is a start.