Suggestion: cookie.set could return value instead of this about keen-tracking.js

[dustinlarimer]

@mijamo that's a fair question and suggestion. The purpose for returning this for modification tasks is that it makes these methods chainable:

const cookie = Keen.utils.cookie('user_id')
  .options({ ... })
  .expire(14)
  .set({
    // ...
  })

If you tack another .get('user_id') after .set() I think you would get the same result that you're looking for:

const userID = sessionCookie.get('user_id') || sessionCookie.set('user_id', Keen.helpers.getUniqueId()).get('user_id')

Would that do the trick?

[mijamo]

Yes this is what I ended up doing. I understood the chainable necessity for options or expire but a bit less for set which in my mind was always at the end of the chain, but I also understand it may be useful in some cases.

[dustinlarimer]

thanks @mijamo!