Add option to apply rabbit hole behaviors to all users including administrators

Hi, I have opened MR with the fix.

I used the patch as the base but did a small naming change, fixed the issue with the checkbox value not being saved and it is capable with the latest dev version.

Here is the patch:
https://git.drupalcode.org/project/rabbit_hole/-/merge_requests/25.patch

@gilmord thanks! Can you please change a target branch to 2.0.x? 8.x-1.x is for bug fixes only.

Rebase is required now, I will check it later

New features will go into 2.0.x

In a first step, this would be helpful to at least see, as admin, that there's a rabbit hole active: ✨ Add message for admins to indicate what action would happen Fixed

This needs a manual rebase! :)

@Anybody I did not know what branch has to be used as a base, so I used the one the fork created for me.
And it was an outdated branch, and the fresh one has a lot of changes.

Instead of rebasing we should consider reworking it, should be faster than rebasing from an outdated version.

However you like! Thanks

+1 for getting this patch working with the latest code, rebase, re-edit whatever is easiest :)

Since the base branch got switched after the MR was created, and 8x is different than 2x I am super confused how to rebase this. Anyone else better at git foo than me?

Ok, this one is non-trivial and it wasn't my git foo. This patch needs to be reworked because the underlying methods are quite a bit different between version 1 and 2.

Here's a patch of the latest MR to make testing easier

The merge request works perfectly! Thanks for the contribution. My team is really looking forward to this feature.

@Greg Boggs thanks for the manual testing. I started from a bit different approach, but after giving it a second thought, I think the original idea of no_bypass property is good and will play nicely along with ✨ Add message for admins to indicate what action would happen Fixed .

It'll be committed after adding some tests. I'll work on it tomorrow.

Matroskeen → closed merge request !25

Editing the issue summary

It should be good to go. The title and description of new checkbox might be improved, so if you have any suggestions - feel free to open a follow-up task. Thanks everyone!

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