Overview

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

Probably an existing issue but weird that testInaccessibleFieldsInAutoSave doesn't assert anything about path in the response since that is the field that inaccessible, is it in the response for this user or not? Shouldn't asserting it is not included? Maybe we should add that here to make sure we didn't change the behavior? I checked and it is not in the response

Proposed resolution

User interface changes

Issue fork experience_builder-3532496

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

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 » larowlan
Status: Needs review » Reviewed & tested by the community

LGTM, don't even think this needs to run by Wim.

wim leers made their first commit to this issue’s fork.

  • wim leers committed dc74e53c on 0.x authored by larowlan 
    Issue #3532496 by larowlan, penyaskito: Assert fields the user can't...
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Assigned: larowlan » Unassigned
Priority: Normal » Minor
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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