[FOF Terms] Optional consents / Opt-Out

GreXXL

rafaucau thanks, great proposals. I have activated bounties on this.

rafaucau

rafaucau Additionally, it would be useful to have a configurable webhook that triggers when a user opts in or out of a consent.

I cant't edit first post (no permissions), so I'm adding here that this for example would allow the use of a tool such as Make or Zapier to integrate with newsletter subscription and unsubscription.

rafaucau

clarkwinkelmann You could leave the old acceptance status and send a notification / email to users that the privacy policy has been updated, and the user can optionally review changes and in the profile settings can change the acceptance status.

AngryDragonite

Working on it.

AngryDragonite

PR has been submited, and is ready to be review by the fof/terms team.
FriendsOfFlarum/terms51

IanM

rafaucau AngryDragonite

Sorry this has taken so long, @davetodave178 looked over this recently and I've taken a look this weekend. On the whole I think we're good to go, but I do need a minor change before this is merged (FriendsOfFlarum/terms51)

Once this is done, I'll give it a spin locally, and if everything checks out I will merge & release 🙂

Thanks for your patience so far!

rafaucau

@GreXXL @clarkwinkelmann @IanM Any chance of a review of this pull request in the near future?

GreXXL

rafaucau I will try to reach somone from @FriendsOfFlarum. Please understand that this is a project not operated by Flarum staff.

akrimony

Test community

rafaucau

PR has been ignored for many months. Can someone else from the FOF team review it?
FriendsOfFlarum/terms51

GreXXL

rafaucau does this new version satisfy the requirements you laid out in the bounty? I think it would be interesting for @AngryDragonite so he can claim his bounty now that it has been merged into the FoF Terms extension.

IanM

I've merged and tagged 1.4.0-beta.1 for external testing and to validate amongst the community if this PR fully meets the bounty requirements.

Please give it a spin, and report your feedback here!

composer require fof/terms:"^1.4.0-beta.1"`
php flarum migrate
php flarum cache:clear

rafaucau

GreXXL Yes, @AngryDragonite can now claim the bounty - the implementation meets our requirements. He was working with us when he developed this feature, though he's currently on a break from our team due to busy studies. If any adjustments are needed, I'll submit those changes myself. I'll be testing the terms consent integrations with other extensions this week.