- π§πͺBelgium dieterholvoet Brussels
I would make
noindex
a base field instead of a separate table. I'll try to work on this. - π§πͺBelgium dieterholvoet Brussels
I propose not porting this to Drupal 8 anymore, since it has been long unsupported. We could consider supporting Drupal 9 since it hasn't been unsupported for that long.
- Status changed to Needs review
10 months ago 12:36pm 5 March 2024 - π§πͺBelgium dieterholvoet Brussels
Finished! Some changes I did:
- The module now uses a base field for storage. This also removes the need for a custom Views integration.
- I wrote a new README.md following the current template.
- I renamed the permission to be more clear about what it does.
- I changed some labels/descriptions to be more clear.
I think this is ready to be published. We should start a
2.0.x
branch for the D9/D10 version, rebase the MR here on that branch and merge it. - π§πͺBelgium dieterholvoet Brussels
I changed the destination branch to 2.0.x. @gisle should I wait for your review before merging/releasing this?
- π³π΄Norway gisle Norway
No, don't wait for my review. I am busy with another project, and shall not be able to do a timely review at the present time.
-
DieterHolvoet β
committed 3565e5e2 on 2.0.x
Issue #2513054 by DieterHolvoet, arknoll: Drupal 9/10 port
-
DieterHolvoet β
committed 3565e5e2 on 2.0.x
- Status changed to Fixed
10 months ago 12:33pm 6 March 2024 - π§πͺBelgium dieterholvoet Brussels
@gisle any chance you can make 2.0.x the default branch in Gitlab? I don't seem to have the necessary permissions.
- π§πͺBelgium dieterholvoet Brussels
Thanks. I created a beta release, I'll wait for a couple months and if no issues arise I'll create a stable release.
Automatically closed - issue fixed for 2 weeks with no activity.