I am searching for co-maintainer(s) of the 7.x and 8.x branches. Please let me know that you are interested. Then we can talk about the further development.

I suggest that we setup a Google Hangout session then, to share information about the further development process. Work on the documentation or user interface or whatever (feel free to be creative) would be also appreciated.

Comments

jepSter created an issue. See original summary.

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented
Title: Seeking co-maintainer(s) » Seeking co-maintainer(s) (7.x and 8.x branches)
Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

It would be really appreciated, if anybody could take care of the 7.x branch. I haven't done there anything since months and there is a lot of things which could be back ported from 8.x to 7.x.

prathibhab.cdac’s picture

prathibhab.cdac CreditAttribution: prathibhab.cdac commented

Yes I am interested in the development part. How can I get involved?

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

Grab an issue, work on it and show up that you are worth to be a maintainer. This could be a good one as start: https://www.drupal.org/node/2899428.

prathibhab.cdac’s picture

prathibhab.cdac CreditAttribution: prathibhab.cdac commented

AccessCheckInterface appears in PermissionsByTermController class. Its used in handleviews and handlenode funtion. How to handle this part?

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

@prathibhab.cdac: Just replace the parameter hinting with the class instead of it's interface. That's easy, since there is only 1 interface per 1 class.

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

@prathibhab.cdac: Please communicate further in the related ticket: https://www.drupal.org/node/2899428

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

Please check the issue Automated Tests, if you like to contribute.

dakku’s picture

dakku CreditAttribution: dakku commented

I am also happy to contribute, will rustle up a patch for your consideration. Hopefully that will be worthy of co-maintainer candidate.

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented

It would be nice if you provide some Kernel Tests. There are currently none and the further development would profit if we enforce that against regression and easier refactoring. Beside that I have written a Behat tests project for PbT: https://github.com/jepster/permissions-by-term-behat

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented
Related issues: +#2904093: Automated Tests
Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented
Related issues: +#2882311: unauthorized access to terms bypassing autocomplete

Can anybody take care of this issue: https://www.drupal.org/node/2882311. It has somewhat of a major priority.

Peter Majmesku’s picture

Peter Majmesku
Location 🇩🇪Düsseldorf
CreditAttribution: Peter Majmesku commented
Status: Active » Closed (outdated)