Skip re-fetching deleted installedPkgs status

[antgamdia]

Description of the change

This PR adds a check before getting the installed pkg status: "if there is not an active subscription, skip the process". This way, when unmounting components, like when we are deleting a package (we get redirected to the apps view), the fetch won't happen.

Benefits

If the app had been deleted, the fetch would have failed, showing an unpleasant error back to the user.

Possible drawbacks

N/A

Applicable issues

• fixes Deleted applications showing an error after deletion #5205

Additional information

N/A

[netlify]

✅ Deploy Preview for kubeapps-dev canceled.

Built without sensitive environment variables

Name	Link
🔨 Latest commit	58a282e
🔍 Latest deploy log	https://app.netlify.com/sites/kubeapps-dev/deploys/62f522fddbc13c00086cae5e

[castelblanque]

Is this really blocking the release?
I'm wondering if we could leave it for the next one (patch release?).
In this way, there will be less risk for breaking new stuff. Plus the UI manual testing has already been done, and we should do another one to check regressions. WDYT?

[antgamdia]

Is this really blocking the release?

For me, this is pretty bad, enough to block the release. I mean, see the UX right after deleting...

Note that the function getInstalledPkgStatus is only being called at getResources, which is something scoped to the installed packages view. Worst case regression is an installed package not changing its status from "reconciling" to "ready"... but that's all. However, leaving it as is would have implications for each pkg we delete...