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

SensioLabs Insight support #331

Closed
wants to merge 11 commits into from
Closed

SensioLabs Insight support #331

wants to merge 11 commits into from

Conversation

Vrtak-CZ
Copy link
Contributor

closes #306

For user uuid and apiToken go to https://insight.sensiolabs.com/account

@espadrine
Copy link
Member

Thanks!

We usually rely on a short (usually 1-letter) disambiguator for each vendor that provides more than one service, such as /npm/v/ for versions. Does SensioLabs provide other services?

Other than that, it looks good to go!

(I will be inaccessible for the week-end, a cousin has a wedding, but I'll work on merging this on monday.)

@Vrtak-CZ
Copy link
Contributor Author

Yes SensioLabs provide more other services. So I can change URL to /sl/i/ or /sensiolabs/i/?

espadrine and others added 3 commits December 22, 2014 22:32
WordPress is spelled with a capital P, just wanted to fix that. Thanks!
@espadrine
Copy link
Member

I have not managed to get it working, even though I created an app and inserted its id and secret information in secret.json.

Have you been successful? Is there something I may be doing wrong?

@Vrtak-CZ
Copy link
Contributor Author

yes everything works on my machine. Maybe you put your app uuid to secret.json instead user uuid.

{
    "sl_insight_userUuid": "user uuid",
    "sl_insight_apiToken": "secret"
}

@espadrine
Copy link
Member

That is indeed the case, but I can't see where to fetch the user uuid.

@espadrine
Copy link
Member

I figured it out, thanks to you. Merged! Will be live in 24h.

1ec5aea

@espadrine espadrine closed this Dec 23, 2014
@Vrtak-CZ
Copy link
Contributor Author

THX

@bkdotcom
Copy link
Contributor

bkdotcom commented Jan 9, 2018

switching from SesionLab's badget to shields.io badge....
I kinda liked the ★'s in the sensio version...
would it be possible to optionally use the star char (U+2605)?
and/or "SensioLabs" instead of "check" on the left?

@paulmelnikow
Copy link
Member

This is a very old issue. Could you open a new one, please?

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.

Add support for SensioLabsInsight
7 participants