Dispatch ChangeSet instead of creating new EditorState on resetView in CodeBlock component

[laurakwhit]

Description & motivation 💭

We're seeing the occasional RangeError: Invalid change range [num] to [num] (in doc of length [num]) error coming from @codemirror and it looks like it's happening for Codeblocks in Modal components. Specifically after this is called ⬇️

ui/src/lib/components/workflow/client-actions/signal-confirmation-modal.svelte

Line 28 in 50a5e10

codeBlock?.resetView(input);

This PR dispatches a transaction with a ChangeSet in resetView on the CodeBlock component instead of creating a whole new EditorState and calling setState.

Screenshots (if applicable) 📸

Design Considerations 🎨

Testing 🧪

How was this tested 👻

• Manual testing
• E2E tests added
• Unit tests added

Steps for others to test: 🚶🏽‍♂️🚶🏽‍♀️

• On a Running workflow
  • select Send a Signal > fill out Name and Input > close the modal
  • select Send a Signal again
    • Verify Input has been cleared
  • fill out Name > tab to Input and fill out (this is where error was occurring)
    • Verify no error occurs and signal can be submitted successfully

Checklists

Draft Checklist

Merge Checklist

Issue(s) closed

Docs

Any docs updates needed?

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
holocene	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 30, 2024 9:38pm