[Bug] app-ng7-mat-breadcrumb component not picking up on breadcrumb updates about ng7-dynamic-breadcrumb HOT 9 CLOSED

I hotfixed this issue resetting breadcrumb with this.dynamicBreadcrumbService.updateBreadcrumbLabels([]); in ngDestroy hook.

from ng7-dynamic-breadcrumb.

Note I was able to code a work around by having the ApplicationsComponent set the breadcrumb for the child. This shouldn't be necessary but seems to prove to me that there are more than one breadcrumb service instance being loaded. And the ApplicationSummaryComponent is setting a different one somehow. I have no idea how this happened.

from ng7-dynamic-breadcrumb.

This will reproduce the issue https://stackblitz.com/edit/ng7-mat-breadcrumb-6nkaqh
Click back and forth between the pageOneId 1 and 2 links on the main screen and watch the breadcrumbs on the next page.

from ng7-dynamic-breadcrumb.

@canpan14 could you please tell me what the workarround is? until this bug gets fixed?

from ng7-dynamic-breadcrumb.

shouldn't the module implement forRoot for this to work?

from ng7-dynamic-breadcrumb.

That's might help. I didn't try forking this and recoding it yet, a little busy (sorry for the late reply).

Here is a version with the workaround. Note this is a little sloppy, but the important thing to note is that I'm calling update breadcrumb in the page.component.ts. Which updates the breadcrumbs on the next page for me.
https://stackblitz.com/edit/ng7-mat-breadcrumb-e4cyks

from ng7-dynamic-breadcrumb.

Found out how to fix it. Please note I only did it for the material angular one to show as an example.
I left comments in the pull request, let me know if you need anymore information.

from ng7-dynamic-breadcrumb.

Hello, do we have any updates on this issue? Are there any problems with PR, when could it be merged?

from ng7-dynamic-breadcrumb.

New Version unable to reproduce, We Believe issue has been fixed.

from ng7-dynamic-breadcrumb.