Overview

See https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...

In #3529622: Make auto-save manager only support passing and returning entity objects, remove support for arbitrary data, use deterministic hashing to ensure entries are actually updates we now perform validation on the entity rather than on arbitrary array values. This means the validation of the converted component tree reports the error against the XB field name rather than 'model'.

We should translate that.

Proposed resolution

User interface changes

Issue fork experience_builder-3532487

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

larowlan created an issue. See original summary.

larowlan opened merge request !1194

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Active » Needs review
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾 Seville 💃 Andalusia, UTC+2 🇪🇺
commented
Assigned: Unassigned » wim leers
Status: Needs review » Reviewed & tested by the community

Looks pretty straightforward. Surprised it didn't affect more tests.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: wim leers » Unassigned
Priority: Normal » Minor
Issue tags: +validation

LGTM!

  • wim leers committed b5dfcee8 on 0.x authored by larowlan 
    Issue #3532487 by larowlan, penyaskito: ApiAutoSaveController should...
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +Usability

Status: Fixed » Closed (fixed)

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