search

sleroy / sonar-slack-notifier-plugin

SonarQube plugin for sending notifications to Slack
Other
40 stars 17 forks source link

Feature request : show letter ratings instead of number #4

Open Indigo744 opened 4 years ago

Indigo744 commented 4 years ago

Hello,

I have a feature request: would it be possible to display the proper rating in letter (like A, B, C...) instead of numbers in the notification?

Right now, it is displayed as number:

Reliability Rating on New Code: OK
1, error if >1

But SQ uses letters: image

So the idea would be to display something along this:

Reliability Rating on New Code: OK
A, error if > A

Thank you.

sleroy commented 4 years ago

OK, let me see how to handle that.

Sylvain

Le lun. 23 déc. 2019 à 15:23, Guillaume notifications@github.com a écrit :

Hello,

I have a feature request: would it be possible to display the proper rating in letter (like A, B, C...) instead of numbers in the notification?

Right now, it is displayed as number:

Reliability Rating on New Code: OK 1, error if >1

But SQ uses letters: [image: image] https://user-images.githubusercontent.com/7137528/71363025-0f4a3680-2598-11ea-909f-f5ecde7e30c3.png

So the idea would be to display something along this:

Reliability Rating on New Code: OK A, error if > A

Thank you.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/sleroy/sonar-slack-notifier-plugin/issues/4?email_source=notifications&email_token=ABL7XEV4ZE63ILFDQAP2WBLQ2DCWBA5CNFSM4J6UWID2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICK3D7Q, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABL7XEUBEVUZMZHZIR6UEC3Q2DCWBANCNFSM4J6UWIDQ .

trintott251 commented 4 years ago

Hi, Great plugin, works perfectly for me. I have a question that i think is related to OP's question so i'll post it here. In addition to change the numbers to letters, can i change the message all together? So for instance show how many code smells or bugs are found in stead of just stating that it has passed or failed? Cheers

sleroy commented 3 years ago

To do that, I would need to integrate the SonarWS like with the fork of the SonarPDF plugin https://github.com/sleroy/sonar-pdf-report.

However, I don't know if my plugin is actually used or if the other users are handling the Slack functionality with another plugin.