Skip to content
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

Add instruction to generate an access token. #4

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

Autive
Copy link
Contributor

@Autive Autive commented Feb 21, 2024

I was struggling to integrate this package cause in my mind the access token was pa_{Identification Code}.
Just later i discovered that creating a client make the needed access token, so adding an instruction maybe helps others.

@Autive
Copy link
Contributor Author

Autive commented Feb 22, 2024

@Kugelschieber maybe it would also be a big plus to mention "access tokens" in the Pirsch dashboard.
For me creating Clients to get an access token isn't really that obvious. So having an additional text similar to:

"Create clients to create an access token or oAuth credentials to access the API.
Your dashboard ID is XygJrQzg3w "

@Kugelschieber
Copy link
Member

Thank you for the suggestion :) We can add it to the hint on the left.

@zepfietje
Copy link
Collaborator

I've made some final tweaks, but thanks for the PR, @Autive!

@zepfietje zepfietje merged commit e918ffb into pirsch-analytics:master Mar 5, 2024
19 checks passed
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.

3 participants