Skip to content

feat: create a workflow test agent, websocket api, and UI#21

Draft
thantos wants to merge 4 commits intomainfrom
sussman/test_harness
Draft

feat: create a workflow test agent, websocket api, and UI#21
thantos wants to merge 4 commits intomainfrom
sussman/test_harness

Conversation

@thantos
Copy link
Copy Markdown
Contributor

@thantos thantos commented Nov 17, 2022

No description provided.

@@ -0,0 +1,150 @@
import { activity, eventual } from "@eventual/core";
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love this example - great job.

Let's give it a better name than src/tester haha.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha, thanks, sure.

@thantos thantos marked this pull request as draft November 17, 2022 18:36
"index.js",
Buffer.from(result.outputFiles![0]!.contents)
.toString("utf-8")
.replace("[WEBSOCKETURL]", apiStage.url)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you do this replacement using an esbuild define, so the token is inserted in the bundling step?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants