Skip to content

Fix setting Geo sync capacity in the admin panel

Nick Thomas requested to merge (removed):fix-setting-geo-capacity into master

What does this MR do?

Add some missing parameters to the controller params filter, refactoring to DRY the list up.

Are there points in the code the reviewer needs to double check?

The responsibility for parameter filtering gets split between controller and service like this, but it's a step better than the logic being duplicated.

Why was this MR needed?

Without it, setting max capacity via the admin panel is broken. This is a feature to be introduced in %10.2

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports