Sub-module report is not getting the api key

Created on 21 January 2022, almost 3 years ago
Updated 15 March 2023, almost 2 years ago

Problem/Motivation

The sub-module "Sendgrid Integration Reports" is not working with the API key implementation.

Steps to reproduce

1. Install the following modules: sendgrid_integration, sendgrid_integration_reports, key
2. Create API key and assign the key in Sendgrid settings page.
3. Now visit the report page "admin/reports/sendgrid". You will see no data available.

Proposed resolution

Create a patch to fix the bug.

πŸ› Bug report
Status

Needs review

Version

2.0

Component

Code

Created by

πŸ‡¦πŸ‡ΊAustralia finaukaufusi Sydney

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡³πŸ‡±Netherlands idebr

    This is still an issue on the 8.x-2.x branch.

    Attached patch syncs the implementation of settings the API key between \Drupal\sendgrid_integration\Plugin\Mail\SendGridMail and \Drupal\sendgrid_integration_reports\Api, so the sendgrid.settings.apikey is interpreted as an API key even with the Key module installed.

  • Status changed to RTBC almost 2 years ago
  • πŸ‡³πŸ‡±Netherlands anneke_vde

    I tested patch #5, when applied I see data at "admin/reports/sendgrid".

  • Status changed to Needs review almost 2 years ago
  • πŸ‡ΊπŸ‡ΈUnited States Perignon

    I would re-test with the latest dev code. This patch is out of date. Also just before this patch is this code which does the same thing:

    if ($this->moduleHandler->moduleExists('key')) {
          $key = \Drupal::service('key.repository')->getKey($key_secret);
          if ($key && $key->getKeyValue()) {
            $this->apiKey = $key->getKeyValue();
          }
        }
        else {
          $this->apiKey = $key_secret;
        }
Production build 0.71.5 2024