upcoming: [M3-8328] - Add Analytics Events to Linode Create v2

[bnussman-akamai]

Description 📝

• This PR adds custom analytics events to Linode Create v2
• This PR's goal is to copy over only events UX wanted to retain from Linode Create v1
  • View this doc to see what events are kept / not kept 📄
  • The events that are retained should work the same way as they did on Linode Create v1

How to test 🧪

Prerequisites

• Enable Linode Create v2 feature flag 🇺🇸
• Run _satellite.setDebug(true) in your browser's console ⌨️
• Refresh the page 🔄

Verification steps

• Create various Linodes on different tabs of the Linode Create flow
  • Verify Analytics events fire for each creation
  • Verify those analytics events match what Linode Create v1 would capture

As an Author I have considered 🤔

• 👀 Doing a self review
• ❔ Our contribution guidelines
• 🤏 Splitting feature into small PRs
• ➕ Adding a changeset
• 🧪 Providing/Improving test coverage
• 🔐 Removing all sensitive information from the code and PR description
• 🚩 Using a feature flag to protect the release
• 👣 Providing comprehensive reproduction steps
• 📑 Providing or updating our documentation
• 🕛 Scheduling a pair reviewing session
• 📱 Providing mobile support
• ♿ Providing accessibility support

[github-actions]

Coverage Report: ✅
Base Coverage: 82.23%
Current Coverage: 82.23%

[mjac0bs]

Thanks! I confirmed sendLinodePowerOffEvent, sendApiAwarenessClickEvent, sendLinodeCreateFlowDocsClickEvent, and sendLinodeCreateEvent were still firing and the other events are no longer being tracked.

Aside from sendLinodeCreate on Clone, everything fired as expected.

[mjac0bs]

🚢

[abailly-akamai]

Confirming seeing proper events being fired ✅

[abailly-akamai]

can we avoid this assertion?

[bnussman-akamai]

I wish. Not sure why Typescript can't see the values.linode check in the if statement

[abailly-akamai]

I think the type checker does not refine the type of values.linode inside the lambda function passed to queryFn.
just make it a new const

if (type === 'Clone Linode' && values.linode) {
    // @todo use Linode Query key factory when implemented
    const linodeId = values.linode.id;
    const linode = await queryClient.ensureQueryData({
      queryFn: () => getLinode(linodeId),
      queryKey: ['linodes', 'linode', linodeId, 'details'],
    });

    sendCreateLinodeEvent('clone', values.type, {
      isLinodePoweredOff: linode.status === 'offline',
    });
  }

[abailly-akamai]

Should we still send an event if we don't have a label?

[bnussman-akamai]

v1 does, so I opted to send even for undefined. I had to update the type because v1 just lied about types