Preserve decimal Customer.io timestamps (#3656)

* Preserve decimal Customer.io timestamps

* update comment to mention decimal fractional timestamps

---------

Co-authored-by: Dr.J <[email protected]>
Co-authored-by: Joe Ayoub <[email protected]>
Co-authored-by: sgwcollins <[email protected]>

Author: 4 people

packages/destination-actions/src/destinations/customerio/__tests__/utils.test.ts

@@ -1,4 +1,4 @@
-import { resolveIdentifiers, isIsoDate } from '../utils'
+import { convertValidTimestamp, resolveIdentifiers, isIsoDate } from '../utils'
 
 describe('isIsoDate', () => {
   it('should return true for valid ISO date with fractional seconds from 1-9 digits', () => {
@@ -79,3 +79,9 @@ describe('resolveIdentifiers', () => {
     expect(resolveIdentifiers({})).toBeUndefined()
   })
 })
+
+describe('convertValidTimestamp', () => {
+  it('should leave decimal unix timestamps unchanged', () => {
+    expect(convertValidTimestamp('1712345678.123')).toBe('1712345678.123')
+  })
+})

packages/destination-actions/src/destinations/customerio/utils.ts

@@ -39,10 +39,11 @@ export enum AccountRegion {
 
 export const convertValidTimestamp = <Value = unknown>(value: Value): Value | number => {
   // Timestamps may be on a `string` field, so check if the string is only
-  // numbers. If it is, ignore it since it's probably already a unix timestamp.
+  // digits (optionally with a fractional part). If it is, ignore it since
+  // it's probably already a unix timestamp (integer or decimal).
   // DayJS doesn't parse unix timestamps correctly outside of the `.unix()`
   // initializer.
-  if (typeof value !== 'string' || /^\d+$/.test(value)) {
+  if (typeof value !== 'string' || /^\d+(\.\d+)?$/.test(value)) {
     return value
   }