Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#311: List more than one example on the Examples page :) #312

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

webchick
Copy link
Contributor

(Corresponding Jira ticket EDU-1842)

What was changed

This PR adds a few more "non-controversial" examples (meaning, limited to the github.com/temporalio namespace for now) to the Examples page, including a reference to the new Order Management System which was just launched at Replay, so we can give people a bit more of a head start.

It also introduces the concept of "categories" on this page, so we can expand the list here to other types of examples in the future.

Why?

See #311 for background/rationale.

Checklist

  1. Pushes forward [Feature Request] Expand the list of examples on the Example applications page #311 (maybe doesn't "close" it)

  2. How was this tested:
    I ran vale index.md as instructed in the README and proactively fixed a few things that it found. (It unfortunately barfs on the link titles to the SDK examples though :)

I couldn't seem to get the link checking working (I blame midnight oil :D) so I did a manual click-through on those.

  1. Any docs updates needed?
    N/A?

@webchick webchick self-assigned this Sep 27, 2024
@webchick webchick requested a review from a team as a code owner September 27, 2024 08:07
Copy link

vercel bot commented Sep 27, 2024

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

Name Status Preview Comments Updated (UTC)
temporal-learning ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 3:28pm

@webchick
Copy link
Contributor Author

A bikeshed we could reasonably paint here btw, is what categories make the most sense for the people perusing this content (vs. how "we" think of them); however, I once again underscore that right now the examples page has only ONE example, so I would prefer to defer that painting party to another time. ;)

@axfelix
Copy link
Contributor

axfelix commented Sep 27, 2024

These are nice additions and the build is passing which is rare for first-time learn or docs site contributors, so thank you very much for that!

I think we've been meaning to have a longer conversation for a while about whether we want to link to the sample repos directly or whether we want to try to progressively write them up as tutorials, so this might be an open-ended addition -- and likewise for the OMS, I don't know how we're planning to socialize that at the moment.

So basically -- this PR should absolutely stay open for now, but I'm not 100% on where we stand on merging it currently.

@axfelix
Copy link
Contributor

axfelix commented Sep 27, 2024

@temporalio/education

@webchick
Copy link
Contributor Author

webchick commented Sep 27, 2024

I think we've been meaning to have a longer conversation for a while about whether we want to link to the sample repos directly or whether we want to try to progressively write them up as tutorials

Makes sense. Here are a few data points to consider:

  1. Right now, it takes a person on average between 4-6 months to go from "I just learned about Temporal" to "I built my first POC using it." That is an awfully long time to get over the suck threshold.
  2. During both discussions internally (with PMM, SAs), as well as externally (with customers + users at Replay), a lack of examples to start from was widely cited as the top reason this tends to happen.
  3. If you do a Google search for "temporal examples," https://learn.temporal.io/examples/ is the top result (nicely done!) so providing people with a strong first impression here feels quite important.

If we link directly to repos as a starting point, we can easily change those links over to tutorials as they become available. But in the meantime, all 3 of the above points are improved.

@axfelix
Copy link
Contributor

axfelix commented Sep 27, 2024

@napcs For your consideration next week while I'm out

@napcs
Copy link
Collaborator

napcs commented Oct 2, 2024

This is the right first step and I appreciate @webchick for taking the time to make this happen.

@napcs napcs merged commit b52e807 into temporalio:main Oct 2, 2024
4 checks passed
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