Skip to content

Commit

Permalink
Merge pull request #1782 from PrefectHQ/subscription-injection-warnings
Browse files Browse the repository at this point in the history
BugFix: no longer attempting to inject subscriptions that are not needed and never provided
  • Loading branch information
Evan Sutherland committed May 10, 2022
2 parents d9bf81e + 6bfb697 commit 35cbc67
Showing 1 changed file with 1 addition and 7 deletions.
Expand Up @@ -20,7 +20,7 @@

<script lang="ts" setup>
import { useSubscription } from '@prefecthq/vue-compositions'
import { computed, inject as vueInject } from 'vue'
import { computed } from 'vue'
import BreadCrumbs from '@/components/BreadCrumbs.vue'
import DeploymentPanel from '@/components/DeploymentPanel.vue'
import FlowPanel from '@/components/FlowPanel.vue'
Expand All @@ -36,7 +36,6 @@
import { inject } from '@/utilities/inject'
import { showPanel } from '@/utilities/panels'
import { toPluralString } from '@/utilities/strings'
import { deploymentsListSubscriptionKey, flowsListSubscriptionKey } from '@/utilities/subscriptions'
const props = defineProps<{ flow: Flow }>()
Expand All @@ -60,9 +59,6 @@
const deploymentsCountSubscription = useSubscription(deploymentsApi.getDeploymentsCount, [countFilter])
const deploymentsCount = computed(() => deploymentsCountSubscription.response ?? 0)
const flowsListSubscription = vueInject(flowsListSubscriptionKey)
const deploymentsListSubscription = vueInject(deploymentsListSubscriptionKey)
function openFlowPanel(): void {
showPanel(FlowPanel, {
flow: props.flow,
Expand All @@ -79,8 +75,6 @@
dashboardRoute: route,
deploymentsApi,
flowRunsApi,
flowsListSubscription,
deploymentsListSubscription,
})
}
</script>
Expand Down

0 comments on commit 35cbc67

Please sign in to comment.