refactor: quota modal usage

[JammingBen]

Description

Before, the state of the quota modal was handled by the outside where it was being used. This caused quite some boilerplate code because it's effectively always the same.

Refactors the quota modal usage so that the modal state handling is done via the quota action instead, meaning the consuming party doesn't need to care about it anymore.

Related Issue

• refs Refactor modal #10095

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Technical debt
• Tests

[update-docs]

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

[dschmidt]

We will have this logic in a lot of places in the future I assume - maybe worth adding a helper function for this?

[kulmann]

Hate it (the if function then call else use value), that was exactly the reason why I went for always the callback when I adjusted the SideBarPanel interface...

Helper would be ok-ish, but the helper might also advocates such type definitions. I'm also fine without the helper. 🤷

[JammingBen]

Then let's make it always a function, I need to rebase anyway.

[sonarqubecloud]

Quality Gate failed

Failed conditions

30.77% Condition Coverage on New Code (required ≥ 50%)

See analysis details on SonarCloud