Description

The Siteimprove plugin bridges the gap between Drupal and the Siteimprove Intelligence Platform.
You are now able to put your Siteimprove results to use where they are most valuable – during your content creation and editing process.
With analytics and content insights always at hand, contributors can test, fix, and optimize their work continuously .
Once the detected issues have been assessed, you can directly re-recheck the relevant page and see if further actions are needed. Delivering a superior digital experience has never been more efficient and convenient.

Project Url

https://www.drupal.org/project/siteimprove

Project Git Url

git clone --branch 8.x-1.x https://git.drupal.org/project/siteimprove.git

Installation and configuration

Siteimprove Plugin can be installed like any other Drupal module. Place it in the modules directory for your site and enable it on the `admin/modules` page.
Visit the Siteimprove Plugin settings page and regenerate auth token if you need it.

Manual reviews of other projects

https://www.drupal.org/node/2867080#comment-12025943
https://www.drupal.org/node/2866223#comment-12025999
https://www.drupal.org/node/2825145#comment-12026018
https://www.drupal.org/node/2867664#comment-12027927
https://www.drupal.org/node/2866797#comment-12030891
https://www.drupal.org/node/2868153#comment-12030930
https://www.drupal.org/node/2687695#comment-12031249
https://www.drupal.org/node/2657506#comment-12031361

Comments

cbccharlie created an issue. See original summary.

PA robot’s picture

Multiple Applications
It appears that there have been multiple project applications opened under your username:

Project 1: https://www.drupal.org/node/2866393

Project 2: https://www.drupal.org/node/2866392

As successful completion of the project application process results in the applicant being granted the 'Create Full Projects' permission, there is no need to take multiple applications through the process. Once the first application has been successfully approved, then the applicant can promote other projects without review. Because of this, posting multiple applications is not necessary, and results in additional workload for reviewers ... which in turn results in longer wait times for everyone in the queue. With this in mind, your secondary applications have been marked as 'closed(duplicate)', with only one application left open (chosen at random).

If you prefer that we proceed through this review process with a different application than the one which was left open, then feel free to close the 'open' application as a duplicate, and re-open one of the project applications which had been closed.

I'm a robot and this is an automated message from Project Applications Scraper.

PA robot’s picture

Status: Needs review » Needs work

There are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpsgitdrupalorgprojectsiteimprovegit

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

cbccharlie’s picture

Status: Needs work » Needs review
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
Issue tags: +PAreview: review bonus
cbccharlie’s picture

Issue summary: View changes
harsh.behl’s picture

Hi cbccharlie,

I have found that you have generated a new token in install file using your servie 'siteimprove.utils'. And in readme.txt I have found:
Who can use this plugin?
The plugin requires a Siteimprove subscription to be used.
Signup for a [FreeTrial](https://siteimprove.com/account/create “Free trial”)
to test it out.

when I goto /admin/config/system/siteimprove page I found you have already generated token. I think this token must be generated after the subscription or there needs to be a check before generating a new token weather the user has an account with siteimprove.

Again this is a thought or suggestion, let me have your opinion.

Thanks :)

cbccharlie’s picture

Hi pen,

The token ID and the Siteimprove suscription are different aspects and there is no relation between them.

By one side, Siteimprove needs to assign an UniqueID to each one of the websites that use the pluging; this is why when the siteimprove module is installed on your Drupal application, a unique ID is automatically obtained and assigned to it. Of course, if you want you can change that Token ID (accessign to the form you wrote), but it is not needed if it has assigned one value.

By other side, you will need a siteimprove account, but there is no relation at all between the account and the token ID. Once you have the module installed, you can simple log-in into siteimprove.

Thank you for your comments!

cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Issue summary: View changes
cbccharlie’s picture

Hi @pen,

Have you found any bugs that are application blocker? Is it possible to pass the application to RTBC or there is something that needs to be worked on?

Thanks!

cbccharlie’s picture

Priority: Normal » Major
logan.H’s picture

Can see redundant code im these functions siteimprove_form_node_form_alter() and siteimprove_form_taxonomy_term_form_alter()

and in siteimprove_entity_insert() and siteimprove_entity_update().

Those codes can be moved into a reusable function and call it from hook_form_HOOK_form_alter() alters

cbccharlie’s picture

Hi @logan.H,

Thank you for your comments. I already did those changes. Anything else?

Thanks!

cbccharlie’s picture

Priority: Major » Critical
matroskeen’s picture

Hello guys, what is the purpose of this issue?

Looks like Siteimprove is already covered by the security advisory policy. It has stable releases and it's great!

I think we may mark this issue as "Fixed", how do you think?

cbccharlie’s picture

Hi @matroskeen,

Yes, you are right.

Thanks!

cbccharlie’s picture

Status: Needs review » Fixed
cbccharlie’s picture

Status: Fixed » Closed (fixed)