? ? ? Pending

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
16 Apr 2021

As an partial alternative to #33165

Summary of Changes

The Sample Data module is now hidden if no Sample Data is available to be installed.

Testing Instructions

Disable all Sample Data Plugins
Check Home Dashboard

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

None

avatar Bakual Bakual - open - 16 Apr 2021
avatar Bakual Bakual - change - 16 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Apr 2021
Category Administration Language & Strings Modules
avatar brianteeman brianteeman - test_item - 16 Apr 2021 - Tested successfully
avatar brianteeman
brianteeman - comment - 16 Apr 2021

I have tested this item successfully on 84a7cfd


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

avatar PhilETaylor
PhilETaylor - comment - 16 Apr 2021

so if the other pr just unpublished the plugins, and we leave the module enabled, and this pr hides the module, does that make everyone happy or should I just abandon the other PR? I'm not invested in it, it was just requested by @dgrammatiko

avatar Bakual
Bakual - comment - 16 Apr 2021

I did another PR to make unpublishing sample data plugins easier. Personally I'd prefer my approach - but I have no say.

avatar dgrammatiko
dgrammatiko - comment - 16 Apr 2021

I did another PR to make unpublishing sample data plugins easier.

But the problem is that the UX is ruined whenever users need to manually complete tasks that should be transparent to them. Basically, this whole conversation just proves that the whole architecture (plugin+module) here is totally wrong. The sample data should have been a component (eg com_onboard) that will be the default route whenever a user installs something (and it provides an onboarding experience). Also, it would be nice because it would allow devs to do whatever they want there (SPA basically) and then just remove the routing (or even uninstall the component) so the default route defaults back to com_cpanel...

avatar Bakual
Bakual - comment - 16 Apr 2021

Feel free to rewrite and improve the whole thing for J5 ?
The current implementation never was proposed as a perfect solution. But even like this it is far better than the hardcoded, unmaintainable SQL files we had before ;-)

avatar Quy Quy - test_item - 16 Apr 2021 - Tested successfully
avatar Quy
Quy - comment - 16 Apr 2021

I have tested this item successfully on 84a7cfd


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

avatar Quy Quy - change - 16 Apr 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 16 Apr 2021

RTC


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

avatar infograf768 infograf768 - close - 17 Apr 2021
avatar infograf768 infograf768 - merge - 17 Apr 2021
avatar infograf768 infograf768 - change - 17 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-17 05:47:17
Closed_By infograf768
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 17 Apr 2021

tks

Add a Comment

Login with GitHub to post a comment