Skip to content
This repository has been archived by the owner on Mar 3, 2022. It is now read-only.

added support for extra/custom query params with test #355

Merged
merged 1 commit into from
Sep 23, 2017

Conversation

btimo
Copy link
Contributor

@btimo btimo commented Jun 19, 2017

Solution for issue #315.

Spec are as followed:

  • only accepting object
  • default value is {}
  • can set initial value
  • can get value
  • can set new value
    • if value not an object, set to default value {}
  • (key,value) couples are added to the end of the url

@brockallen
Copy link
Member

Looks pretty good -- I'll merge when I can sit down and really go over it. Thanks!

@btimo
Copy link
Contributor Author

btimo commented Jul 23, 2017

UP 🙃

@skolmer
Copy link

skolmer commented Sep 10, 2017

This will be really useful to us, thanks. 👍 How about allowing this on the signout request too?

@brockallen brockallen added this to the 1.4.0 milestone Sep 23, 2017
@brockallen brockallen merged commit 147c5d8 into IdentityModel:dev Sep 23, 2017
@brockallen
Copy link
Member

Thanks!

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

Successfully merging this pull request may close these issues.

None yet

3 participants