Skip to content

Fixing Temporal.Duration and Temporal.Instant usage in use cases and demos#732

Open
pattishin wants to merge 8 commits into
mainfrom
refactor/update-temporal-guides
Open

Fixing Temporal.Duration and Temporal.Instant usage in use cases and demos#732
pattishin wants to merge 8 commits into
mainfrom
refactor/update-temporal-guides

Conversation

@pattishin
Copy link
Copy Markdown
Contributor

@pattishin pattishin commented May 6, 2026

Related to: #374 , #375 , #377 (overall #79)

@sffc I believe I captured your comments from #374 and #375, but please let me know if any were overlooked!

@pattishin pattishin requested a review from sffc May 6, 2026 23:15
@pattishin pattishin marked this pull request as ready for review May 6, 2026 23:17
@pattishin
Copy link
Copy Markdown
Contributor Author

pattishin commented May 11, 2026

Adding in changes based on comments from #376 - converting back to draft.

@pattishin pattishin marked this pull request as draft May 11, 2026 14:58
@rviscomi
Copy link
Copy Markdown
Member

@pattishin will this be ready for review before launch?

@pattishin
Copy link
Copy Markdown
Contributor Author

@pattishin will this be ready for review before launch?

@rviscomi I can make it happen! I won't add in changes for #376 just yet. I'll ping when it's up!

}
prevDateStateValue = dateState.deadline;
}
dateValue.textContent = dateState.deadline.toLocaleTimeString([], { hour: '2-digit', minute: '2-digit' });
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.

This was just formatting issues. No actual content modified.

@pattishin pattishin marked this pull request as ready for review May 18, 2026 23:14
@pattishin
Copy link
Copy Markdown
Contributor Author

@rviscomi Ready for review again!

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.

2 participants