Thank you for this great module! It would be great to move this project towards stable, and have it work with Drupal 9. I would be happy to help out, including trying to get the issue queue cleaned up.

Checklist

  • Link to project: Real-time SEO for Drupal
  • Record of owner agreement: no
  • Maintainers: JSON
  • Year of last Drupal.org activity of maintainers:
    - Open Social: 2020 (owner)
    - Kimiez: 2019
    - Jaap Jan Koster: 2020
    - Bram ten Hove: 2020
  • Project Status:
  • Record of attempts to PM owner and maintainers:
    2020-09-05: Open Social PM by mandclu
    2020-09-28: Open Social PM by gisle
    2020-09-28: Jaap Jan Koster PM by gisle
    2020-09-28: Bram ten Hove PM by gisle
    2020-09-30: Kimiez PM by kiamlaluno
  • Project covered by Drupal’s security advisory policy: Yes
  • Permission to opt into Drupal’s security advisory policy: Yes

Comments

mandclu created an issue. See original summary.

mandclu’s picture

Project: Real-time SEO for Drupal » Drupal.org project ownership
Version: 8.x-2.x-dev »
Component: Miscellaneous » Co-maintaining offer
Category: Support request » Task
Issue summary: View changes
gisle’s picture

Issue summary: View changes
Status: Active » Postponed

The present owner and two of the maintainers are still around. We need to hear their opinion about this. I've sent a PM to all three asking them to comment here.

Postponing for now. Please set back to “Active” if there is no response from anyone in 14 days.

gisle’s picture

Issue summary: View changes
avpaderno’s picture

Issue summary: View changes

I contacted also Kimiez, since Kimiez can add new co-maintainers/maintainers and logged in in the last 12 months.

kingdutch’s picture

Hello,

Thanks for the message. We're discussing internally what the best way to move forward here is.

~ Alexander

phjou’s picture

It's currently impossible to install the module on Drupal 9, the last patch for that has been waiting for a merge since July.
#3119603: Drupal 9 readiness

It is blocking lots of people to use the module since you have to make a fork if you want to use it. I hope it get resolved soon.
Thanks for your help.

mandclu’s picture

Status: Postponed » Active

Moving this back to active, as it's been 14 days since the last comment. Would love to help make the excellent functionality available in Drupal 9

gisle’s picture

Assigned: Unassigned » gisle
Status: Active » Fixed

No answer from any of the project's maintainers in 14 days (Kingdutch, answered - sort of - but he is only a co-maintainer).

Patch to make project compatible with Drupal 9 exists, but has not been merged. I think enough time has elapsed for internal discussions and I am moving this forward. mandclu has been added as a maintainer.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

kingdutch’s picture

I have been the primary maintainer of this module for the past three years. So I take some offense in being labeled only a co-maintainer. This has also been the view from Open Social’s standpoint as organisation.

We now realise that my role in the project may not be marked properly (I have all the permissions I need to do my work, so this is not something we’ve ever considered). As Open Social employee (you can find my picture on that page), I am allowed to speak on behalf of Open Social for this module.

This also explains why I had not received any communication before or about this issue (until Kimmiez asked me to reply here after she received her PM). The Open Social account is an account that is used for project ownership and for security reasons is not easily (or regularly) accessible.

Jaap Jan has left our company quite some time ago, which is probably why he won’t respond. I’ll ask he be removed from the maintainer list.

We’ve since (two weeks before your comment) merged all open RTBC issues and ensured Drupal 9 compatibility (the Drupal 9 issue for 1.x has been re-opened since and closed today due to a bug that was missed in the initial fix/review).

We’d prefer seeing someone active in the issue queue before taking them on board as (co-)maintainer. My quick searches (using “followed by” or “submitted by”) did not reveal such activity for mandclu. Additionally module maintenance and improvement requires more than is available on Drupal.org as external JavaScript is required that can not be hosted on Drupal.org due to licensing.

We’re committing to making sure we review the issue queue on a bi-weekly basis going forward to ensure community submitted issues are merged. If any more urgent work is needed, I am easily reachable on the Drupal.org Slack under the same nickname (as are most of my colleagues).

mandclu’s picture

I mean no offence and genuinely only wanted to help. I will confess that after this issue was resolved I did notice that a Drupal 9-compatible release had been created, and the issue queue worked on already.

It's a fair point that more active involvement in the issue queue would be optimal prior to being added as a co-maintainer, particularly when there are active maintainers in place. I'm glad to see this module being maintained as I would like to use it more, and will likely be more involved in the issue as a result, over time.

@KIngdutch if you feel it's more appropriate to remove me as a co-maintainer, I will gladly defer to your judgement,

avpaderno’s picture

The users who are contacted in these cases are the project maintainer, the maintainers (users who have all the permissions on the project), and any co-maintainer (users who don't have all the permissions on the project) who can add new co-maintainers/maintainers. I apologize if we forgot to contact somebody.

For what I recall, at the time we contacted users, we contacted all the users who should be contacted.

gisle’s picture

I have been the primary maintainer of this module for the past three years. So I take some offense in being labeled only a co-maintainer. This has also been the view from Open Social’s standpoint as organisation.

No offence was intended.

The term "co-maintainer" is a technical term that is defined on this page: https://www.drupal.org/node/251466

I checked the permissions carefully before contacting the maintainers, and at the time, you did not have the permission to add new maintainers or co-maintainers to the project. I.e. you were not in a position to resolve this co-maintaining offer. The procedure for handling these requests only requires the webmaster to contact and wait for a response from the owner in these cases. Contacting other maintainers is not required, but is a matter of courtesy at the discretion of the webmaster. It is OK to have a a corporate account as project owner, but if that is done, you should make sure that the email-address associated with the account is monitored, as it will be used for important communication about your project.

The webmaster expects co-maintainer requests to be resolved by the project's owner or one of the project's maintainers (i.e. a person that has the permission to add a co-maintainer). The time line we use assumes that requests are resolved 14 days after the owner and maintainers have been contacted. By "resolving" I mean that either the person offering to co-maintain is added as co-maintainer, or that a definitive statement declining the offer is posted in response to the offer.

Your answer in comment #6 did not resolve the matter. It is bad for the Drupal ecosystem that our projects are not well maintained (in this case a patch for Drupal 9 existed when the issue was brought to the attention of the webmasters on Sep 28, but it was not committed until October 6). In such cases it is standard procedure for the webmasters to intervene and resolve the matter in the absence of any definitive action to resolve it from the project's owner or his/her appointed deputies.

We’re committing to making sure we review the issue queue on a bi-weekly basis going forward to ensure community submitted issues are merged.

The patch required by Drupal 9 became RTBC on April 24 2020. There was no review of it from any Open Social employee until Oct 6 2020. That indicates that a bi-weekly review is not done.

kingdutch’s picture

Thanks for the clear explanation Gisle, those are some things I haven't encountered in my time here on Drupal.org.

We've now fixed the permissions so that I'm clearly marked as maintainer. I will also raise the point internally that we're missing important communication through the corporate account and need to rectify that.

> @KIngdutch if you feel it's more appropriate to remove me as a co-maintainer, I will gladly defer to your judgement,

We have decided to remove you for the time being. I will update the project page and the open roadmap issue in the coming 14 days to indicate what the 2.0 stable release has been stuck on.

I will also open an issue in the issue queue to propose dropping support for the Drupal 7 version as we haven't had any Drupal 7 projects internally for some time and the changes made for the 2.0 version are difficult to backport to Drupal 7 (hence the lack of release since 2017). Being able to close all the open 7.x-1.x issues that won't get fixed will also help in clearing up the issue queue.

We welcome your feedback in those issues. If there's anything you would like to discuss in real-time or have any urgent matters about the Real-Time SEO module, please don't hesitate to contact me through the Drupal Slack. We strive to make sure an issue such as this one is not needed in the future.

gisle’s picture

I will also open an issue in the issue queue to propose dropping support for the Drupal 7 version as we haven't had any Drupal 7 projects internally for some time and the changes made for the 2.0 version are difficult to backport to Drupal 7 (hence the lack of release since 2017).

The Drupal 7 branch of the project still has more than 7000 installations. Drupal 7 does not reach EOL until November 2022.

Dropping support for the Drupal 7 branch of the project before November 2022 will be bad for our ecosystem. May I suggest some alternatives:

  1. Add an external maintainer to the project to maintain the Drupal 7 branch only. You can do it here (Component "Needs maintainer"). I have done that with some projects to keep the branch I am not interested in myself (because I have no clients using the branch) maintained. I've never had any bad experiences (such as these external maintainers interfering with the branch my company is in charge of) when doing this.
  2. If you don't want the Drupal 7 branch associated with Open Social after you've dropped support, fork the Drupal 7 branch into a separate project, and ask the webmasters to assign ownership to "Unsupported projects". That way, it will no longer be associated Open Social, and will automatically be available for adoption by anyone who wants to keep on maintaining it.
kingdutch’s picture

Thanks for the suggestions Gisle! That's why I wanted to open up an issue rather than do it outright. I will include your suggestions in the discussion.