Digital-Alchemy-TS / Digital-Alchemy-TS/synapse

Timestamp sensors do not accept dayjs as state

Open
#122 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
3
Forks
1
PR merge metrics
No merged PRs in 30d

Description

## 🐜 Bug description

Want to create a sensor to track timestamps of an event, but it seems like the functionality to take in `dayjs()` objects for sensors was never implemented.

Throws some typing errors, but on the internals it looks like implementation was started.

## 🎙️ Additional details

Probably a fixme item?

Contributor guide

Open the contributing guide

Research direction

No files or tests are named. Start by tracing the sensor state types and the existing timestamp handling, then reproduce the reported TypeScript errors with a dayjs() value. Done means dayjs objects are accepted as sensor state without the reported typing errors.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.