Problem/Motivation

After saving a node and being redirected to the frontend the user needs to be able to see the latest changes even though the frontend static build is not done. In order to make this possible, add ?auth=1 parameter to url.

Issue fork lupus_decoupled-3314800

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

ciprian.stavovei created an issue. See original summary.

fago’s picture

fago
Primary language German
Location Vienna
commented
Assigned: Unassigned » ciprian.stavovei
useernamee’s picture

useernamee commented

There's a consideration to add a new concept for this issue to make solution more efficient.

(Bad scenario: If after saving a node, you're not redirected to the node, then you're not able to see the latest changes).

fago’s picture

fago
Primary language German
Location Vienna
commented

let's implement what we have working and consider improvements as 2nd step, so we have something working quick.

ciprian.stavovei opened merge request !21

ciprian.stavovei’s picture

ciprian.stavovei commented
Assigned: ciprian.stavovei » useernamee
Status: Active » Needs review

Added implementation for this. Ready for review.

useernamee’s picture

useernamee commented

small change is needed.

I'm not sure how much opinionated towards specific fe this is. But it is not a big change, so when PR is improved, it could go into RTBC and be merged.

useernamee’s picture

useernamee commented
Assigned: useernamee » ciprian.stavovei
useernamee’s picture

useernamee commented
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Needs work

> I'm not sure how much opinionated towards specific fe this is.
I think it's ok, every frontend can add support for the paramter.

as I understand MR needs work, thus marking so.

Finally, this auth parameter must be added for preview routes also. Let's add a follow-up for this.

ciprian.stavovei’s picture

ciprian.stavovei commented
Status: Needs work » Needs review

I addressed the PR remarks and added follow-up for preview url at #3322382: Add auth=1 query parameter in preview url.

useernamee’s picture

useernamee commented
Status: Needs review » Reviewed & tested by the community

I guess it would make sense to join these two tickets, but the code looks good.

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

petar_basic’s picture

petar_basic commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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