Eppo-exp / node-server-sdk

Eppo Node.js SDK
MIT License
9 stars 0 forks source link

Use common poller #47

Closed aarsilv closed 7 months ago

aarsilv commented 7 months ago

🎟️ Ticket: FF-1471 - Have Node SDK use updated Common JS Module

Motivation and Context

So that our client-side JavaScript SDK can benefit from the polling improvements done to this SDK, we're moving poller to js-client-sdk-common.

👀 I recommend checking out that repository's PR #35 - Common configuration requestor/poller as part of reviewing this one.

Description

Removes poller and instead adjusts this SDK to leverage it within js-client-sdk-common. This greatly simplifies this SDK 📈

How has this been tested?

Leveraged the previously added unit tests in index.spec.ts