Problem/Motivation

Currently we use an event subscriber that is triggered on every request in order to show a simple Drupal message, indicating the environment the user is in. Doing this on every request is a bit much, since we don't do anything else then displaying a message, so its not the best performance wise. Ideally we show only trigger this code when the user is interacting with the UI.

Steps to reproduce

Proposed resolution

Remove the event subscriber and move the code into a service, then call this service when the users is actually interacting with the UI.

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork acquia_connector-3312634

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

nkoporec created an issue. See original summary.

nkoporec opened merge request !18

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Assigned: nkoporec » Unassigned
Status: Active » Needs review
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Issue summary: View changes

  • japerry committed 071c128 on 3.x 
    Issue #3312634 by nkoporec: Refactor CodeStudioMessage event subscriber

mglaman credited japerry.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Fixed

Closing, thanks!

  • japerry authored ea3d816 on 4.x 
    Issue #3312634 by nkoporec: Refactor CodeStudioMessage event subscriber...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.