fix: metrics view model dropdown

[royendo]

Fix missing olap for MotherDuck,
Fix populate table in visual editor properly

Add: open the current metrics view but have the others available but not opened

Checklist:

• Covered by tests
• Ran it and it works as intended
• Reviewed the diff before requesting a review
• Checked for unhandled edge cases
• Linked the issues it closes
• Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
• Intend to cherry-pick into the release branch
• I'm proud of this work!

[AdityaHegde]

Wont this block updating model to other connectors? We should probably allow updating to anything, but update fields like connector when they are selected.

[royendo]

I personally dont think we should support swapping a Duck -> CH or CH -> MD metrics view; (the blank metrics view option is kind of confusing as it starts in an errored state)

Since we do have that option today you can list all connectors but once you select; makes sense to me to be locked down. @ericokuma thoughts?

If we dont agree, i can strip out that part of the logic as MD is broken today and needs to be fixed

[royendo]

I went ahead and removed the filterConnector logic and replaced with defaultExpand, basically visually taking the user to the connector that is currently being used.

Im still a bit against it but i also realize a misclick of connector would lock them out;

[royendo]

https://www.loom.com/share/5d2823e3cdd544faae84c690a4c23c42?from_recorder=1&focus_title=1

on another hand, do we even need model selector in the UI if the table lists all and metrics view already has all OLAP in the dropdown 🤔

[ericokuma]

If we don't show model selector, then why create a model?

[ericokuma]

Also, this still doesn't solve for: "the blank metrics view option is kind of confusing as it starts in an errored state"

[royendo]

If we don't show model selector, then why create a model?

Rill model = duckdb table/view; the PR screenshot shows duckdb as one of the selectable OLAP,

Not for this PR but just an idea as I was poking around

[royendo]

Also, this still doesn't solve for: "the blank metrics view option is kind of confusing as it starts in an errored state"

This is also out of scope, simple fix is remove blank metrics view from the UI.

Don't see a real reason to keep this path now that Aditya added selectable metrics based on connector