Problem/Motivation

If the remote status is not set, an error occurs after ingesting the widgets and trying to add a new one.

Steps to reproduce

Ingest widgets, without the status set, and try to add a new one.

Proposed resolution

Set the unknown status whenever a new widget is about to ingest and the status doesn't exist.

Remaining tasks

Set unknown by default.

User interface changes

N/A

API changes

N/A

Data model changes

There wont be a null remote status, just unknown or the status set in the registry.

Comments

isholgueras created an issue. See original summary.

isholgueras’s picture

Here is the patch to set 'unknown' as default remote status in the ingestion process.

The test probably will fail because it's using a constant added in this issue and the constant is not yet defined. At least until 3337158 is merged.

isholgueras’s picture

Assigned: isholgueras » Unassigned
Status: Needs work » Needs review
e0ipso’s picture

Status: Needs review » Needs work

We need to update the dependency version in the composer.json once the Widget Type release is out.

e0ipso’s picture

Priority: Critical » Normal

We need to update this patch to depend on widget_type 1.5.14.

isholgueras’s picture

Status: Needs work » Needs review
StatusFileSize
new1.13 KB

Here is the patch with the composer require on widget_type:"^1.5.14"

isholgueras’s picture

I didn't have the branch 1.x updated. Here is the right patch

  • e0ipso committed 76a5847c on 1.x authored by isholgueras
    Issue #3339913 by isholgueras, e0ipso: Set 'unknown' as default remote...
e0ipso’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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