[PP-1] Use actual page URL instead of hardcoding the query string in unlock URL

Created on 25 September 2024, 3 months ago

Problem/Motivation

Follow-up to GUI install multiple modules at once. Needs work ,

    const div = document.createElement('div');
    if (err.unlock_url && err.unlock_url !== '') {
      div.innerHTML += `<p>${errorMessage} <a href="${
        err.unlock_url
      }&destination=admin/modules/browse">${Drupal.t(

@phenaproxima suggested

This assumes that the unlock URL has a query string. IMHO we would benefit here from more robust parsing of the actual URL, but that can be done in a follow-up.

Steps to reproduce

Proposed resolution

Instead of adding hard coded destination=admin/modules/browse", parse and use actual page url.

📌 Task
Status

Active

Version

2.0

Component

Code

Created by

🇮🇳India narendraR Jaipur, India

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Production build 0.71.5 2024