[MLv2] [FE] Migrate ViewHeader to MLv2

[ranquild]

ViewHeader

Place 1

metabase/frontend/src/metabase/query_builder/components/view/ViewHeader.jsx

Line 96 in b14003c

const filtersCount = question.query().filters().length;

In MLv2 it could be:

const query = question._getMLv2Query();
const previousQuery = usePrevious(query);

useEffect(() => {
  // existing code

  const filtersCount = Lib.filters(query, -1).length;
  const previousFiltersCount = Lib.filters(previousQuery, -1).length;

  // existing code
});

Place 2

metabase/frontend/src/metabase/query_builder/components/view/ViewHeader.jsx

Line 110 in b14003c

isStructured && question.query().topLevelQuery().hasAggregations();

In MLv2 it could be:

Lib.aggregations(query, -1).length > 0

For non-mbql queries aggregations would be an empty list.

-1 means the last stage of the query.

Place 3

metabase/frontend/src/metabase/query_builder/components/view/ViewHeader.jsx

Line 419 in b14003c

const isNewQuery = !question.query().hasData();

MLv2:

const isNewQuery = Lib.sourceTableOrCardId(query) == null;

Tasks

Beta Give feedback

1. Migrate ViewTitleHeader to MLv2 #37375
   .Team/QueryingComponents no-backport +2
   
2. Migrate ViewTitleHeaderRightSide component

[nemanjaglumac]

Migrated steps 1 and 2 from the original issue description.
The only outstanding reference to the MLv1 in this file is (step 3):

metabase/frontend/src/metabase/query_builder/components/view/ViewHeader.jsx

Line 419 in b14003c

const isNewQuery = !question.query().hasData();

but we cannot fully migrate it until we start supporting native queries with MLv2.

cc @NevRA