Add migration doc draft. #91
Conversation
Shivs11
left a comment
Shivs11
left a comment
There was a problem hiding this comment.
I did a pass and left some comments
|
@drewhoskins-temporal could you take a look at this? I think this doc should be designed/evaluated in the context of our other "communication" plans. @robholland my broad comment is that I think for the Temporal Worker Versioning concepts, we should be linking out to the temporal docs liberally vs redefining things (kind of what Shivam said). I think this is a good first step though! |
Refocus migration guide on unversioned -> versioned.
Co-authored-by: Carly de Frondeville <[email protected]>
carlydf
left a comment
carlydf
left a comment
There was a problem hiding this comment.
Another review!
Main points in the review are:
- Env vars are now compatible with SDK worker-options-loading library here, so we should direct people to set their workers up that way for ease.
- I don't think we should recommend Manual rollout strategy for people's first rollout, but if you feel differently we should discuss!
Clean up last few places that recommend Manual
carlydf
dismissed
drewhoskins-temporal’s stale review
Review is addressed (temporal worker versioning docs are linked, and concepts will be moved to temporal docs after merging + linked from here to temporal docs after they are moved to temporal docs), and I want to merge this PR asap
5 participants
Fixes #34