New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix token name #46

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@MrVincs

MrVincs commented Jan 27, 2018

On previous commit, the token option 'name' can set the token name in response.

But if the token it's on data.primary.secondary.token the name of token can't be the same value.

That's why, i want to add 'path' option to token option 馃憤

MrVincs added some commits Jan 27, 2018

Add 'path' to token option
Add token path option on README
Set the default path value
set path value : .token
Add token path option
Add token path to data response

pi0 added a commit that referenced this pull request Feb 1, 2018

@pi0

This comment has been minimized.

Member

pi0 commented Feb 1, 2018

Thanks for your PR. It will be possible using Auth 4.x, endpoints.xx.propertyName option using dotprop. PR #50.

@pi0 pi0 closed this Feb 1, 2018

pi0 added a commit that referenced this pull request Feb 2, 2018

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