Data Flows: cannot edit an operation

[mslonina]

Describe the Bug

Hi,

first of all - long awaited feature with DataFlows - awesome!

But, once the flow is created, and an operation is added, it looks like, we cannot edit the operation, but we can replace it entirely i.e. Log in to console cannot be updated, after clicking Edit icon, the dialog with list of operations appears.

To Reproduce

1. Create a flow
2. Add a operation - Log in to console and save it
3. Try to edit Log in to console operation - the list of operations appears, not the operation itself

Errors Shown

None

What version of Directus are you using?

9.12.0

What version of Node.js are you using?

Docker

What database are you using?

Docker

What browser are you using?

Safari

How are you deploying Directus?

Docker

[vanling]

Workaround would be to refresh when you are in editing screen.

rec.mov

[rijkvanzanten]

I'm not able to reproduce this locally under the dev build, lemme try a prod instance

[rijkvanzanten]

Can confirm. This only happens in the prod build 🤦🏻

[rijkvanzanten]

oh. and it only happens in prod Safari 🙈

[vanling]

I got in edge :) Versie 101.0.1210.53 (Officiële build) (arm64)

[rijkvanzanten]

Ah yeah I edited the trigger, not the operation 👍🏻

[vanling]

Maybe timing thing between route/url update and opening the dialog/drawer. Dev build slower?

[rijkvanzanten]

I think these refs aren't correctly starting on the values of the prop due to some timing issue. The drawer should only be rendered if the prop exists, but for some reason there's a small discrepancy in timing between prod and dev of when the prop exists (I think)

directus/app/src/modules/settings/routes/flows/components/operation-detail.vue

Lines 99 to 102 in 6e87d8a

	const options = ref<Record<string, any>>(props.operation?.options ?? {});
	const operationType = ref<string | undefined>(props.operation?.type);
	const operationKey = ref<string | null>(props.operation?.key ?? null);
	const operationName = ref<string | null>(props.operation?.name ?? null);

[mslonina]

Thanks! I can confirm it works now!