Skip to content

fix: Consolidate resource and Table dropdown#9074

Merged
royendo merged 1 commit intomainfrom
consolidate-resource-tables-dropdown
Mar 18, 2026
Merged

fix: Consolidate resource and Table dropdown#9074
royendo merged 1 commit intomainfrom
consolidate-resource-tables-dropdown

Conversation

@royendo
Copy link
Copy Markdown
Contributor

@royendo royendo commented Mar 17, 2026

Action When shown When reconciling
Describe Always Enabled
View Logs Always Enabled
View Partitions Partitioned models Enabled
--- separator ---    
Refresh Errored Partitions Models with errors Disabled + tooltip
Full Refresh Models & Sources Disabled + tooltip
Incremental Refresh Incremental models only Disabled + tooltip
Screenshot 2026-03-17 at 10 39 40 Screenshot 2026-03-17 at 10 39 25

Changes:

  • ActionsCell (resources page): Added isReconciling prop, onViewPartitionsClick prop, View Partitions menu item, disabled refresh items with tooltip
    instead of hiding the whole dropdown
  • ModelActionsCell (tables page): Added isReconciling prop, disabled refresh items with tooltip "Model is currently being reconciled"
  • ProjectResourcesTable: Passes isReconciling and onViewPartitionsClick, added ModelPartitionsDialog
  • ModelsTable: Passes isReconciling derived from isResourceReconciling()

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!

@royendo royendo requested a review from ericpgreen2 March 17, 2026 14:42
@royendo royendo merged commit 5344cdd into main Mar 18, 2026
17 of 18 checks passed
@royendo royendo deleted the consolidate-resource-tables-dropdown branch March 18, 2026 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants