Skip to content

[5.3] [Events] Use event classes for Privacy plugins#43602

Merged
laoneo merged 9 commits intojoomla:5.3-devfrom
Fedik:plg-event-privacy
Feb 5, 2025
Merged

[5.3] [Events] Use event classes for Privacy plugins#43602
laoneo merged 9 commits intojoomla:5.3-devfrom
Fedik:plg-event-privacy

Conversation

@Fedik
Copy link
Member

@Fedik Fedik commented Jun 2, 2024

Summary of Changes

Updateprivacy plugins to use SubscriberInterface and an event classes.

Testing Instructions

Check the privacy "export request" still works.

Actual result BEFORE applying this Pull Request

Works

Expected result AFTER applying this Pull Request

Works

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:
  • No documentation changes for docs.joomla.org needed
  • Pull Request link for manual.joomla.org: New Event classes Manual#177
  • No documentation changes for manual.joomla.org needed

@exlemor
Copy link

exlemor commented Jul 24, 2024

I have tested this item ✅ successfully on 347c42e

I was able to test this successfully.

I did a full Privacy Consent user data export cycle.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43602.

 Conflicts:
	plugins/privacy/consents/src/Extension/Consents.php
	plugins/privacy/contact/src/Extension/Contact.php
	plugins/privacy/content/src/Extension/Content.php
	plugins/privacy/message/src/Extension/Message.php
@exlemor
Copy link

exlemor commented Aug 24, 2024

I have tested this item ✅ successfully on 37e2fa4

Hi Fedik, it behaved the same before and after the PR Patch and the emails arrives in both cases.

That said, the link provided below in the email does not bring you to a page that you can enter the token in both cases if that matters.

  1. Visit the following URL: https://pbf11.joom.ws/privacy-component/confirm?confirm_token=8917a9dc760171cdfcf4c24345fcb838

  2. Copy your token from this email, visit the referenced URL, and paste your token into the form.
    URL: https://pbf11.joom.ws/privacy-component/confirm
    Token: 8917a9dc760171cdfcf4c24345fcb838


    This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43602.

@HLeithner
Copy link
Member

This pull request has been automatically rebased to 5.3-dev.

@HLeithner HLeithner changed the base branch from 5.2-dev to 5.3-dev September 2, 2024 08:51
@HLeithner HLeithner changed the title [5.2][Events] Use event classes for Privacy plugins [5.3] [Events] Use event classes for Privacy plugins Sep 2, 2024
@Hackwar Hackwar removed the PR-5.2-dev label Sep 3, 2024
@laoneo laoneo merged commit c6158f4 into joomla:5.3-dev Feb 5, 2025
3 checks passed
@laoneo
Copy link
Member

laoneo commented Feb 5, 2025

Thanks!

@laoneo laoneo added this to the Joomla! 5.3.0 milestone Feb 5, 2025
@Fedik Fedik deleted the plg-event-privacy branch February 5, 2025 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants