Skip to content

Comments

Allow providing params via a closure#83

Merged
dsrees merged 1 commit intomasterfrom
feature/accept-params-closure
Apr 20, 2021
Merged

Allow providing params via a closure#83
dsrees merged 1 commit intomasterfrom
feature/accept-params-closure

Conversation

@dsrees
Copy link
Owner

@dsrees dsrees commented Apr 20, 2021

  • Building endpointUrl using a closure to determine the params to be sent
  • Changing the class constructor to accept a payloadClosure instead of just a payload
  • Added a new alternate constructor to support the previous constructor which takes a constant or null payload

Closes #82

@dsrees dsrees force-pushed the feature/accept-params-closure branch from d16ce13 to 3019390 Compare April 20, 2021 14:53
@dsrees dsrees merged commit 53265a9 into master Apr 20, 2021
@dsrees dsrees deleted the feature/accept-params-closure branch April 20, 2021 14:54
@lukasz-kalnik-gcx
Copy link

Wow, this was super fast! Thank you for reacting so quickly. Any information on when/what version it will be released in?

@lukasz-kalnik-gcx
Copy link

Ok, I see version 0.5.0 is in the works. Looking forward to trying it out 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tries to reconnect with invalid token

2 participants