Remove unwanted dependency on toolbar module brought back by accident

Created on 15 February 2024, 4 months ago
Updated 1 March 2024, 4 months ago

Problem/Motivation

@johnpicozzi reported in Slack this:

Quick question here, I see the MR and issue to decouple navigation from Toolbar. However I'm getting a message on my status report that indicates I have the toolbar module installed. Which is true, however it is still a requirement for the navigation module. I'm thinking we may have missed removing it from navigation.info.yml. Is that accurate or am I missing something?

Checked the source code and found it was certainly TRUE.

Reviewed back the issue and MR for 📌 Decouple from Toolbar module Needs review and the dependency was actually removed from the navigation.info.yml file.
However, as part of Create the Top Bar Needs review the dependency was moved from the bottom to the top of the list.

So, when merging both branches into 1.x, module ended up having the dependency anyway.

Steps to reproduce

Open navigation.info.yml file and confirm that unwanted dependency is there.

Proposed resolution

  • Remove dependency on toolbar module in navigation.info.yml file
  • Confirm that there are no regressions

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Fixed

Version

1.0

Component

Code

Created by

🇪🇸Spain plopesc Valladolid

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @plopesc
  • Status changed to Needs review 4 months ago
  • 🇪🇸Spain plopesc Valladolid

    MR created.

    Please add @johnpicozzi to the credited users list, as he found and reported the bug.

  • Pipeline finished with Success
    4 months ago
    Total: 148s
    #95455
  • 🇺🇸United States johnpicozzi Providence, RI

    Thanks for creating the issue, Will test the MR later today to ensure it resolves the issue. Will report back.

  • Status changed to RTBC 4 months ago
  • 🇮🇳India gaurav_manerkar Vasco Da Gama, Goa

    looks good, can be moved to RTBC.

  • Status changed to Needs work 4 months ago
  • 🇪🇸Spain ckrina Barcelona

    I can disable the Toolbar, but then with Umami Editor and Authors don't have a way to access the administration menus anymore. Not sure if this should be addressed here or in a follow-up.

  • Status changed to RTBC 4 months ago
  • 🇷🇺Russia kostyashupenko Omsk

    It can be a follow-up. I just checked - navigation becomes visible once i'm granting permissions to "Author" role users in Umami.

    Added core task (postponed status).

    Restoring RTBC here

  • Status changed to Fixed 4 months ago
  • 🇪🇸Spain ckrina Barcelona

    Thanks @kostyashupenko! Merging this in then :)

  • Pipeline finished with Success
    4 months ago
    Total: 176s
    #96622
    • plopesc committed c9923730 on 1.x
      Issue #3421505: Remove unnecessary dependency on toolbar module.
  • Automatically closed - issue fixed for 2 weeks with no activity.

  • Pipeline finished with Success
    3 months ago
    Total: 199s
    #136428
  • Pipeline finished with Success
    about 1 month ago
    Total: 214s
    #173492
  • Pipeline finished with Success
    about 1 month ago
    Total: 150s
    #173511
  • Pipeline finished with Success
    about 1 month ago
    Total: 160s
    #179393
  • Pipeline finished with Canceled
    about 1 month ago
    Total: 117s
    #179411
  • Pipeline finished with Success
    about 1 month ago
    #179416
  • Pipeline finished with Success
    about 1 month ago
    Total: 158s
    #179422
  • Pipeline finished with Canceled
    11 days ago
    Total: 188s
    #199154
  • Pipeline finished with Failed
    11 days ago
    Total: 227s
    #199156
  • Pipeline finished with Failed
    11 days ago
    Total: 243s
    #199164
  • Pipeline finished with Failed
    11 days ago
    Total: 380s
    #199177
  • Pipeline finished with Failed
    11 days ago
    Total: 247s
    #199298
  • Pipeline finished with Failed
    11 days ago
    Total: 247s
    #199307
  • Pipeline finished with Failed
    11 days ago
    Total: 241s
    #199322
  • Pipeline finished with Failed
    11 days ago
    Total: 254s
    #199342
  • Pipeline finished with Failed
    11 days ago
    Total: 224s
    #199344
  • Pipeline finished with Failed
    11 days ago
    #199349
  • Pipeline finished with Failed
    11 days ago
    #199359
  • Pipeline finished with Failed
    11 days ago
    Total: 244s
    #199368
  • Pipeline finished with Failed
    11 days ago
    #199386
  • Pipeline finished with Failed
    11 days ago
    Total: 213s
    #199401
  • Pipeline finished with Failed
    11 days ago
    Total: 218s
    #199404
  • Pipeline finished with Failed
    11 days ago
    Total: 222s
    #199416
  • Pipeline finished with Failed
    11 days ago
    Total: 248s
    #199423
  • Pipeline finished with Failed
    11 days ago
    Total: 313s
    #199427
  • Pipeline finished with Failed
    11 days ago
    Total: 227s
    #199430
  • Pipeline finished with Failed
    11 days ago
    Total: 246s
    #199467
  • Pipeline finished with Failed
    11 days ago
    Total: 231s
    #199477
  • Pipeline finished with Failed
    11 days ago
    #199539
  • Pipeline finished with Failed
    11 days ago
    Total: 252s
    #199567
  • Pipeline finished with Canceled
    11 days ago
    Total: 166s
    #199570
  • Pipeline finished with Success
    11 days ago
    Total: 294s
    #199572
Production build 0.69.0 2024