Problem/Motivation

Hi, just playing around with an idea. I'd love to pull the Slack related logic out of the .module file. I think that doing so may remove the need to inject the http client service and a number of dependencies.

Instead, I'd love to a proper service that can have all of the supporting services injected.

Proposed resolution

Refactor the current code to provide service injection.

Remaining tasks

  • Refactor notify_to_slack.module
  • pull logic into a proper service and reference the new service in the .module

Issue fork notify_to_slack-3257364

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

cosmicdreams created an issue. See original summary.

cosmicdreams’s picture

cosmicdreams commented

Ah! Nevermind! Looks like version 2.x already does that.

cosmicdreams’s picture

cosmicdreams commented
Status: Active » Closed (duplicate)