+1 for rolling back the name change + documenting on the project page that the developers and site owners should NOT use the 10.2.0 tagged release nor the 10.0-dev version.
Compared version 10.0.0 and 10.2.0 and this major name change has not been documented with a single line. There is also no issue in the queue covering the renaming as far as I can see. This is really bad practice and causing big headaches for us too, as we have a bigger Canto integration built on top of the Canto Connector module.
I have seen many weird things in my 17 years on d.o, but this definitely end in top 10.
Co-maintainer ship granted. Thanks for helping moving this module forward.
Moving to "Drupal.org project ownership".
Source project: https://www.drupal.org/project/canto_connector β
Right. The official process to get maintainer access is described in the link I posted in my comment:
If the owner does not reply after two weeks, move the issue to the Drupal.org project ownership issue queue, by changing the issue's Project field to Drupal.org project ownership. Please include a link to the project page.
Site moderators will normally try to contact the owner as an extra safeguard unless it is evident the owner is no longer active in the Drupal community, giving to the contacted users two weeks for replying to the message they sent via the Contact tab.
beltofte β created an issue.
Committed. Thanks for reviewing the patch!
Thanks for working on the fix. parse_str() returns either an array or FALSE, and isset() will return TRUE even the variable in argument is FALSE. I have therefore change it from isset() to !empty(). Also implemented a minor change to the foreach loop filtering unwanted query parameters, as $query_parts may not be set.
beltofte β created an issue.
A bit late, but implemented a small wrapper method named filterValidItemIds in ContentEntityFallback which tries to handle the problem. Thanks for reporting this issue.
Thanks all for helping fixing the coding standard issues. I fixed the issues locally with phpcs and also implemented some further adjustments to comments in the code and other things.
Reviewed and merged. Thanks for fixing this issue.
beltofte β made their first commit to this issueβs fork.
Merged. Thanks for fixing the issue!
beltofte β made their first commit to this issueβs fork.
Committed. Thanks for help testing the patch!
@bartvig: Please, look into the temporary fix we discussed and commit it to both versions.
beltofte β created an issue.
Update screenshots and texts to match new version of the Siteimprove overlay UI.
Update screenshot with admin toolbar due to new Siteimprove logo.
Update screenshot + add new bullet with info about the use latest experience checkbox.
Fixed. Included in today's releases.
Committed. Thanks for helping solving the issue. Included in today's releases.
Fixed. Included in today's releases.
Fixed. Included in today's releases.
beltofte β created an issue.
Fixed in both 2.0.x and 8.x-1.x branches. Need backport to 7.x-1.x
Support for Group 3.1.x added in the issue fork. It includes a bit of rewrite of the code in siteimprove_toolbar() too. MR created and assigned to @bartvig.
Notice: The MR is only tested on Group version 3.1.0 and NOT tested on earlier minor or major versions of the module. The code does not really on any API calls or similar to the Group module, but solely "white list" some menu routes and support enabling prepublish for specific Group Types. So it may work on earlier major versions of the Group Module.
Backport: We will not backport this change to the 8.x-1.0 branch, as none of the recommended versions of Group nor the old 8.x-1.5 version supports Drupal 8 or the early minor versions of Drupal 9.
Code changes committed to 8.x-1.x.
Code changes committed to 2.0.x,
Code changes committed to 2.0.x and 8.x-1.x. Thanks for reporting and help fixing the issue.
@bartvig please review and merge.
beltofte β created an issue.
beltofte β created an issue.
Hi Jari,
The issue here is that we have a whitelist of route names where the Siteimprove plugin should show the "This page"-tab (method: input) and fallback to the "Site summary"-tab (method: domain) on all other routes. Right now is whitelist only including entity node and taxonomy_term routes, and not on group routes for example. We will discuss the issue with our Siteimprove contacts and get back to you.
Best regards,
Jens Beltofte