EES-5373: Remove FullTableQuery.BoundaryLevel

[tomjonesdev]

On the back end Admin and Data API services, FullTableQueryRequest/FullTableQuery
are being used by the frond end to execute a table query and optionally specify any boundary level data which should be returned so that for content page map charts, the boundary level data can be fetched and maps can be rendered.

The front end should be responsible for including the BoundaryLevel in the request if boundary level data should be returned, i.e. because boundary level data is needed for rendering a map chart.

This PR moves the property from the query into a separate parameter.

[benoutram]

I think there's an issue to raise here with the rest of the team because it looks like when organising imports Visual Studio is organising them alphabetically and Rider is doing the same but with System imports first.

That adds up to quite a few extra lines to review in PR's.

[tomjonesdev]

What's the rationale for System being first when the other imports are alphabetical? This ticket references the option being added to Rider - the same option is available on VS but is disabled by default.

[benoutram]

I think this will be the default option in Rider, and up until recently all team members used Rider, so a large percentage of the code will have System imports ordered first.

Taken from StyleCop analyzer, the rationale for ordering System imports first is:

Placing all System using directives at the top of the using directives can make the code cleaner and easier to read, and can help make it easier to identify the namespaces that are being used by the code.

I've put this as an item to discuss at the team meeting as it turns out we can try and enforce this an EditorConfig rule one way or the other depending on what we decide.

See https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/dotnet-formatting-options

[benoutram]

Worth checking formatting on this file for long lines.

[tomjonesdev]

I run pnpm format before raising a PR but that doesn't seem to run on .cs files. Afaik VS doesn't have a line length setting because there's a visual word wrapper which avoids changing the actual code/structure.

[benoutram]

Ok I remember that we've chatted about this before. Might be worth chatting to @jack-hive to see if he's found a solution? I've added a note to the team meeting board to discuss the differences between IDE's when formatting long lines.