Use Event Aggregator with Navisworks

[adamhathcock]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 15.86%. Comparing base (67c7ddb) to head (d9e3de2).
Report is 1 commits behind head on dev.

Additional details and impacted files

@@           Coverage Diff           @@
##              dev     #523   +/-   ##
=======================================
  Coverage   15.86%   15.86%           
=======================================
  Files         238      238           
  Lines        4720     4720           
  Branches      578      578           
=======================================
  Hits          749      749           
  Misses       3941     3941           
  Partials       30       30           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jsdbroughton]

Looks legit but I'll test runtime.

Not introduced by this PR but thinking harder about something else about repeated one time binding on send. This is what I was going to test again as mentioned: https://discord.com/channels/726756379083145269/751053204103495711/1332369121203196014

// In NavisworksDocumentEvents.cs
_eventAggregator.GetEvent<IdleEvent>().OneTimeSubscribe(
    nameof(NavisworksDocumentEvents), 
    ProcessModelStateChangeAsync
);

Might want to verify the cleanup behavior with one-time subscriptions if state changes during processing.

Maybe nothing. And not new.

[jsdbroughton]

Successfully tested for non-regressions (including the comment mentioning other issue)