Add localTimestamp and localTimezone #3896
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changelog Entry
Added
localTimestamp
andlocalTimezone
(if available) to all outgoing activities, by @compulim, in PR #3896Description
For all outgoing activities, add
localTimestamp
andlocalTimezone
(if supported by the browser settings).Design
Although we send
localTimestamp
, Azure Bot Services will rename it torawLocalTimestamp
. ThelocalTimestamp
field is same astimestamp
field, which is in UTC.Original activity from Web Chat (captured from network trace)
Activity perceived on bot side
Some environments don't support
window.Intl
Some browsers in some settings, such as IE11 InPrivate mode, will return
undefined
forwindow.Intl
.Specific Changes
postActivitySaga
to filllocalTimestamp
and optionally,localTimezone
dateToLocaleISOString
to create a local timestamp in ISO formatUTC
by default (to reuse the box after last test)CHANGELOG.md
I have updated documentationReview Checklist
Accessibility reviewed (tab order, content readability, alt text, color contrast)CSS styles reviewed (minimal rules, noz-index
)Documents reviewed (docs, samples, live demo)package.json
andpackage-lock.json
reviewedSecurity reviewed (no data URIs, check for nonce leak)