Skip to content

Conversation

@jsundai
Copy link
Contributor

@jsundai jsundai commented Nov 13, 2025

What does this PR do?

https://temporal-documentation-git-tutorial-layout-money-transfe-0fc0c4.preview.thundergun.io/build-your-first-basic-workflow/build-your-first-workflow

Notes to reviewers

files changed include new components for interaction and sdk boxed logos
wip (TODO: make light mode friendly)
aiming for: concise + engaging, dev-to-dev tone, consistent interactions, progressive disclosure.

@jsundai jsundai requested a review from a team as a code owner November 13, 2025 20:00
@vercel
Copy link

vercel bot commented Nov 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
temporal-documentation Ready Ready Preview Comment Dec 1, 2025 3:24pm

@jsundai jsundai changed the title tutorial layout money transfer experimentation build your first basic workflow experimentation Nov 13, 2025
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious why these logos are being added. It looks like all of them are already here.

@flippedcoder
Copy link
Contributor

A couple of general questions.

  • Are we adding all of the UI components because they'll be used on future content like this or is it more specific for this guide? It all looks really great, but I think it looks different from the rest of the site and it threw me off at first.
  • It seemed like this example is already on the learn site. Is this supposed to be migrating the content over or is it supposed to something new?

Most applications require multiple coordinated steps - processing payments, sending emails, updating databases.
This tutorial uses money transfers to demonstrate how Temporal ensures these multi-step processes complete reliably, resuming exactly where they left off even after any failure.
<div style={{textAlign: 'center', margin: '2rem 0'}}>
<img src="/img/moneytransfer/money-withdrawal.png" alt="Money Transfer Application Flow" style={{maxWidth: '100%', height: 'auto'}} />
Copy link
Contributor

Choose a reason for hiding this comment

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

This image is pretty hard to read in light mode.
Screenshot 2025-12-01 at 09 55 02


</SetupSteps>

<details>
Copy link
Contributor

Choose a reason for hiding this comment

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

Nothing wrong here, just a question. I was wondering why we would use the dropdown here. If this is info that we definitely want the user to see it should probably be a regular section so they don't accidentally skip over it.

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree - i am inclined to not collapse this information


<RetryPolicyComparison />

:::important This is a Simplified Example
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor nit: should this be a warning just to highlight this? Totally fine if not!


</SdkTabs>

This code uses a Temporal Client to connect to the Temporal Service, calling its workflow start method to request execution. This returns a handle, and calling result on that handle will block until execution is complete, at which point it provides the result.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This code uses a Temporal Client to connect to the Temporal Service, calling its workflow start method to request execution. This returns a handle, and calling result on that handle will block until execution is complete, at which point it provides the result.
This code uses a Temporal Client to connect to the Temporal Service, calling its Workflow start method to request execution. This returns a handle, and calling result on that handle will block until execution is complete, at which point it provides the result.


**In Terminal 3, start the Workflow:**

The workflow starter script starts a Workflow Execution. Each time you run it, the Temporal Server starts a new Workflow Execution.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The workflow starter script starts a Workflow Execution. Each time you run it, the Temporal Server starts a new Workflow Execution.
The Workflow starter script starts a Workflow Execution. Each time you run it, the Temporal Server starts a new Workflow Execution.

<CallToAction
href="http://localhost:8233"
buttonText="Open the Temporal UI "
description="View the state of the Workflow with the Temporal Web UI"
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we have something that says this is running locally?

- The `withdraw()` Activity completes successfully
- The `deposit()` Activity fails and retries automatically

<AnimatedTerminal
Copy link
Contributor

Choose a reason for hiding this comment

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

I really like this! The only thing is that it makes the page jump when there are new lines in the terminal.

Screen.Recording.2025-12-01.at.11.59.57.mov

Copy link
Contributor

@lennessyy lennessyy left a comment

Choose a reason for hiding this comment

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

This is gorgeous! Great work. My comments are aimed at user-friendliness but may make it less pretty


Before starting this tutorial:

- **Set up a local development environment** for developing Temporal applications
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like the prerequisites are a little unclear to those who are new to our site. I know that if users go through the quick starts if they are set, but if I'm a new user, it's not clear if I am supposed to do the setup AND the quickstarts.

If we don't want to create a separate environment setup page, then we should just remove the two bullets and just tell them to do the quickstarts as a prerequisite.


</SetupSteps>

<details>
Copy link
Contributor

Choose a reason for hiding this comment

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

I agree - i am inclined to not collapse this information

id: build-your-first-workflow
title: Build Your First Workflow
sidebar_label: "Part 1: Build Your First Workflow"
hide_table_of_contents: true
Copy link
Contributor

Choose a reason for hiding this comment

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

i know that the layout is well designed and we could use the vertical space, but i still think this is a long enough tutorial that it'd be beneficial to have the TOC. It's easier for users to make sense of whey are doing in the context of the whole project. Right now, they can't see more than the H2 heading they are in.

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.

4 participants