- Issue created by @matthieuscarset
- 🇺🇸United States phenaproxima Massachusetts
This seems to be squarely in Dashboard's wheelhouse; moving to their issue queue.
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
Closing 📌 Add destination parameter for Edit layout local action Active as duplicated.
- 🇮🇳India koustav_mondal Kolkata
Installed the module and checked in local. After editing the Dashboard layout it is now redirected back to the Dashboard page( admin/dashboard) , not in structure(admin/structure/dashboard). Attaching a video for referance. LGTM+
- Status changed to Needs work
24 days ago 11:40am 5 April 2025 - 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
After a new review this makes lots of sense. I misread the MR.
However, we can just do this in
\Drupal\dashboard\Plugin\SectionStorage\DashboardSectionStorage::getRedirectUrl
Something like this should work.
public function getRedirectUrl() { return Url::fromRoute('entity.dashboard.canonical', ['dashboard' => $this->getStorageId()]); }
Other changes should not be necessary, aside of the added test.
- 🇪🇸Spain penyaskito Seville 💃, Spain 🇪🇸, UTC+2 🇪🇺
This LGTM. Would love @plopesc to verify it too.
- First commit to issue fork.
-
plopesc →
committed 1cbfd7f5 on 2.x authored by
sayan_k_dutta →
Issue #3500368 by sayan_k_dutta, koustav_mondal, penyaskito,...
-
plopesc →
committed 1cbfd7f5 on 2.x authored by
sayan_k_dutta →
Automatically closed - issue fixed for 2 weeks with no activity.