Problem/Motivation

Certain checks depended on the version you are updating to so we should clear readiness result. maybe also when caches are cleared.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tedbow created an issue. See original summary.

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

phenaproxima’s picture

Status: Active » Needs review
tedbow’s picture

Status: Needs review » Needs work

We could test this functionality in build test also to get more complete test coverage.

Create a scenario where
1. ensure cron updates are enabled(I think this the default
2. The site would need to be more than 1 patch update behind
3. assert there is readiness check on an admin page that displays an error that the site is more than 1 patch release behind and will not be able to update via cron.
4. run the update
5 assert the message from 3) is not shown on the same admin page.

I think without the current change 5) would fail

tedbow’s picture

Status: Needs work » Reviewed & tested by the community

  • phenaproxima committed 35fd4e3 on 8.x-2.x
    Issue #3257845 by phenaproxima: Clear readiness stored readiness check...
phenaproxima’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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