? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
29 Jul 2019

Updates the tmpl for the requests view to match J4 standards and styles

To test it will be easier if you have some data. These three rows should cover all the options - obviously replace jos_ with your own db prefix

INSERT INTO `jos_privacy_requests` (`id`, `email`, `requested_at`, `status`, `request_type`, `confirm_token`, `confirm_token_created_at`) VALUES (1, 'one@example.com', '2019-07-29 11:18:14', 1, 'export', '$2y$10$8mubLEGe/27np9z4w3FRSeJDbvR1rLLd0ljelfxKckYge7fE2COdO', '2019-07-29 11:18:14');
INSERT INTO `jos_privacy_requests` (`id`, `email`, `requested_at`, `status`, `request_type`, `confirm_token`, `confirm_token_created_at`) VALUES (2, 'two@example.com', '2019-07-29 11:18:49', 0, 'export', '$2y$10$q91CGg0h1apxAVFJkyJ/xugIZTZ491YPF859Vl5n1uGCzoeIOOb6S', '2019-07-29 11:18:49');
INSERT INTO `jos_privacy_requests` (`id`, `email`, `requested_at`, `status`, `request_type`, `confirm_token`, `confirm_token_created_at`) VALUES (3, 'three@example.com', '2019-05-29 20:34:19', 1, 'remove', '$2y$10$buYqWYjNr5hrL2P374RcA.hM8WtM8Z0NEEsbe.EgMiB9b9J1l14Y6', '2019-05-29 20:34:19');

Other parts of com_privacy are beyond the scope of this PR

@SharkyKZ can you check please if these really are links or should be buttons?

avatar brianteeman brianteeman - open - 29 Jul 2019
avatar brianteeman brianteeman - change - 29 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jul 2019
Category Administration Templates (admin)
avatar brianteeman brianteeman - change - 29 Jul 2019
The description was changed
avatar brianteeman brianteeman - edited - 29 Jul 2019
avatar brianteeman brianteeman - change - 29 Jul 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 29 Jul 2019
Category Administration Templates (admin) Administration Language & Strings Templates (admin)
avatar wilsonge wilsonge - change - 30 Jul 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-07-30 03:02:26
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 30 Jul 2019
avatar wilsonge wilsonge - merge - 30 Jul 2019
avatar wilsonge
wilsonge - comment - 30 Jul 2019

Thanks!

avatar mbabker
mbabker - comment - 30 Jul 2019

can you check please if these really are links or should be buttons?

The first implementation had them as buttons that submitted the standard form for the list view. The JavaScript API couldn't cope with actions that didn't result in a navigation (new page loading, reloading the current view, etc.). So the buttons were changed to "plain" links to bypass the form and the problems that caused. Ref, joomla-projects/privacy-framework@60b8c29#diff-2effc18b36deb5dba68915cd87e0178f

If anyone has any bright ideas, they should be applied because right now a lot of the JavaScript API just does not work outside of the very specific workflow of submitting a form and triggering a navigation action.

avatar brianteeman
brianteeman - comment - 30 Jul 2019

Thanks

Add a Comment

Login with GitHub to post a comment