Problem/Motivation

3.x is being deprecated, we need to add a EOL message to the status report. Once the EOL has passed, we need to disable all API calls to NSPI.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork acquia_connector-3314126

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 !20

nkoporec’s picture

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

mglaman
Primary language English
Location WI, USA
commented
Status: Needs review » Needs work
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented

We need the time checks to respect timezones: 

new \DateTime('Tue Nov 15 2022', new \DateTimeZone('GET TIMEZONE'));
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs work » Needs review
mglaman’s picture

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

Let's check on the timezone parameter. Also, let's use the datetime.time service for getting the current time, so that we can mock it and test these changes.

mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented
Status: Needs work » Needs review
mglaman’s picture

mglaman
Primary language English
Location WI, USA
commented

Updated to not purge data

japerry made their first commit to this issue’s fork.

  • japerry committed 36315db on 3.x authored by nkoporec 
    Issue #3314126 by nkoporec, mglaman: EOL message for 3.x
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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