Modules that set messages aren't shown

Created on 5 February 2025, 9 months ago

Problem/Motivation

Certain modules (TODO need example) set $messages in Drupal when their installer runs. In Project Browser, we should be able to show these messages after the installation completes successfully.

Steps to reproduce

Install a module that sets Drupal messages using Project Browser. (Note, the message is still in queue, so you see it when you navigate away to a new page, but at that point it's too disconnected from what caused the message to be set).

Proposed resolution

I believe we'll need to be able to poll for messages from Drupal after completing the installation task, and then display them using JS.

✨ Feature request
Status

Active

Version

2.0

Component

User experience

Created by

🇺🇸United States chrisfromredfin Portland, Maine

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

Merge Requests

Comments & Activities

  • Issue created by @chrisfromredfin
  • 🇺🇸United States phenaproxima Massachusetts

    We have a clear pathway to doing this now.

    In InstallerController::activate(), as we build the AJAX response, we could pull all messages out of the messenger service and add them to the response with the MessageCommand.

    To test it, all we'd need to do is make TestActivator set a message or two in its activate() method when handing a specific (fake) extension, and then the test would just confirm that those messages show up on the frontend when that fake extension is activated.

    Assigning to @kunal.sachdev for now.

  • 🇮🇳India kunal.sachdev

    kunal.sachdev → made their first commit to this issue’s fork.

  • Merge request !785#3504664 "Modules that set" → (Open) created by kunal.sachdev
  • Pipeline finished with Success
    8 months ago
    Total: 940s
    #453058
  • Pipeline finished with Skipped
    8 months ago
    #453227
  • Pipeline finished with Canceled
    8 months ago
    Total: 132s
    #453257
  • Pipeline finished with Failed
    8 months ago
    Total: 518s
    #453263
  • Pipeline finished with Failed
    8 months ago
    Total: 393s
    #453280
  • Pipeline finished with Failed
    8 months ago
    Total: 384s
    #453290
  • Pipeline finished with Failed
    8 months ago
    Total: 525s
    #455868
  • Pipeline finished with Failed
    8 months ago
    Total: 383s
    #455914
  • Pipeline finished with Failed
    8 months ago
    Total: 613s
    #455938
  • Pipeline finished with Success
    8 months ago
    Total: 511s
    #455959
  • 🇺🇸United States phenaproxima Massachusetts

    This seems really good and straightforward. A few relatively minor suggestions.

  • Pipeline finished with Failed
    8 months ago
    Total: 538s
    #457689
  • Pipeline finished with Success
    4 months ago
    #554086
  • Pipeline finished with Success
    4 months ago
    #554113
  • Pipeline finished with Success
    4 months ago
    #554116
  • Merge request !876Resolve #3504664 "Latest" → (Merged) created by chrisfromredfin
  • Assigned to kunal.sachdev
  • Status changed to Needs work 29 days ago
  • Pipeline finished with Failed
    29 days ago
    #630863
  • Pipeline finished with Failed
    29 days ago
    #630941
  • Pipeline finished with Success
    29 days ago
    #630972
  • Pipeline finished with Success
    28 days ago
    #631319
  • Pipeline finished with Failed
    28 days ago
    #631351
  • Pipeline finished with Failed
    28 days ago
    #631365
  • Pipeline finished with Success
    28 days ago
    #631377
Production build 0.71.5 2024