fix(aci): remove email targetDisplay from UI POST request

[ameliahsu]

we use the targetDisplay to set automatic names for alerts (based on the configured actions), but the POST API doesn't expect a targetDisplay for email actions. this fix strips the targetDisplay field before submitting the form data.

[malwilley]

Is there a test we can add to make sure this doesn't regress in the future?

[malwilley]

Unrelated to this PR, but we should really give action a type here

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
12592	1	12591	10

View the top 1 failed test(s) by shortest run time

AutomationNewSettings creates an automation from scratch and submits successfully

Stack Traces | 4.44s run time

Error: expect(jest.fn()).toHaveBeenCalledWith(...expected)

Expected: Anything, ObjectContaining {"data": {"actionFilters": [{"actions": [{"config": {"targetDisplay": "#alerts", "targetIdentifier": "", "targetType": "specific"}, "data": {}, "integrationId": "1", "status": "active", "type": "slack"}, {"config": {"targetIdentifier": "1", "targetType": "user"}, "data": {}, "status": "active", "type": "email"}], "conditions": [{"comparison": {"key": "env", "match": "co", "value": "prod"}, "conditionResult": true, "type": "tagged_event"}], "logicType": "any-short"}], "config": {"frequency": 1440}, "detectorIds": [], "enabled": true, "environment": null, "name": "Notify #alerts via Slack", "triggers": {"actions": [], "conditions": [{"comparison": true, "conditionResult": true, "type": "first_seen_event"}, {"comparison": true, "conditionResult": true, "type": "reappeared_event"}, {"comparison": true, "conditionResult": true, "type": "regression_event"}], "logicType": "any-short"}}}

Number of calls: 0

Ignored nodes: comments, script, style
...
    at toHaveBeenCalledWith (.../views/automations/new.spec.tsx:154:20)
    at runWithExpensiveErrorDiagnosticsDisabled (.../sentry/node_modules/.pnpm/@[email protected]/node_modules/@.../dom/dist/config.js:47:12)
    at checkCallback (.../sentry/node_modules/.pnpm/@[email protected]/node_modules/@.../dom/dist/wait-for.js:124:77)
    at checkRealTimersCallback (.../sentry/node_modules/.pnpm/@[email protected]/node_modules/@.../dom/dist/wait-for.js:118:16)
    at Timeout.task [as _onTimeout] (.../sentry/node_modules/.pnpm/[email protected]..../jsdom/browser/Window.js:579:19)
    at listOnTimeout (node:internal/timers:588:17)
    at processTimers (node:internal/timers:523:7)

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}