dapr / dapr/js-sdk

Modify actor reminder registration to accommodate possible runtime change

Open
#722 0 comments 0 reactions 0 assignees View on GitHub
area/actors
Dominant language
JavaScript
Stars
217
Forks
104
PR merge metrics
No merged PRs in 30d

Description

## Describe the proposal
HOLD UNTIL CONFIRMED BY RUNTIME

## Describe the feature
[This issue](https://github.com/dapr/dapr/issues/9105) represents a regression in how actors are registered such that an existing reminder is no longer overwritten by default. I've advocated on the thread that the regression implication should be made the norm for this API much as it [was changed](https://github.com/dapr/dapr/issues/8402) on the Jobs API (likely related).

While this has not yet been committed to by the runtime maintainers, this issue should serve as a reminder that the SDK shape around reminder registration may need to be modified in a way that does not introduce a breaking change to accommodate (e.g., to provide an alternative method that throws if the reminder already exists).

## Release Note

RELEASE NOTE:
**ADD** Added method to throw if a reminder already exists on actor instead of strictly overwriting

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.