Removing unload logic from the plugin load logic

[codyrancher]

Summary

The goal is to simplify the logic while possibly improving performance.

fixes #8895

Occurred changes and/or fixed issues

Removed the logic based on this comment in the issue #8895 (comment).

Areas or cases that should be tested

With extensions installed verify the extensions behave as expected wth:

• Logging in
• Logging out
• Navigating between pages

Areas which could experience regressions

See above

Screenshot/Video

No functional/visual changes should be present.

Checklist

• The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
• The PR has a Milestone
• The PR template has been filled out
• The PR has been self reviewed
• The PR has a reviewer assigned
• The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
• The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

[jordojordo]

lgtm - tested with both extension charts and ECI.

[nwmac]

Code looks fine.

I have not tested, but Jordon has.

I think we can remove the code for removePlugin as well - suggest we do this as a follow-on

[codyrancher]

Code looks fine.

I have not tested, but Jordon has.

I think we can remove the code for removePlugin as well - suggest we do this as a follow-on

@nwmac that's actually what I originally did. Unfortunately, it's going to take more digging, without it our built in pkgs weren't loading translations.