-
Notifications
You must be signed in to change notification settings - Fork 6
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: simplified tracking docs #153
Conversation
README.md
Outdated
}); | ||
``` | ||
|
||
## DEPRECATED - Consuming an interaction token |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we just remove this section??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's probably some customers still using it, and if it's not documented here there isn't anywhere else it would be. I'm open to removing it though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm yeah good point, maybe it is worth then just moving it to a different document that is linked from this readme.
Just so that if you are looking for it you can find it, but if you are coming to this for the first time we aren't showing you docs for deprecated methods/solutions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's definitely fair. I'll see if this docs builder we're using supports that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't see a way to easily do that, so I'll actually move this into the documentation on the interactionConsume
method itself.
No description provided.