momentohq / client-sdk-javascript

Official JavaScript SDK for Momento Serverless Cache
Apache License 2.0
55 stars 20 forks source link

feat: remove explicit references to `undefined` for better support of `null` #1413

Closed cprice404 closed 2 weeks ago

cprice404 commented 1 month ago

Users who are consuming our SDK directly from JavaScript (as opposed to TypeScript) may attempt to pass in null in various places where we expose undefined in our APIs. Based on the strict equality checking that we are doing (===), nulls would not be accepted in those places in the API and would result in errors for the users.

This commit applies some eslint rules that @tvald suggested, which forbid the explicit use of undefined as a value, and require the less rigid == equality checks for null (these checks will pass for both null and undefined).

This should allow JS users to pass in null for optional fields in our SDK APIs. I do not believe there are any backward incompatible changes in this PR, nor any functional changes other than being more tolerant of null.

cprice404 commented 2 weeks ago

not sure when this is going to be revisited, closing for now.