Add Configure link

Created on 4 June 2024, 20 days ago
Updated 19 June 2024, 5 days ago

Problem/Motivation

Visit admin/modules and see the agreement module.

Proposed resolution

Changes the agreement.overview route that does not exist to the correct entity.agreement.collection route.

User interface changes

Fixes configuration link!

🐛 Bug report
Status

Fixed

Version

3.0

Component

User interface

Created by

🇮🇳India dev16.addweb

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

Merge Requests

Comments & Activities

  • Issue created by @dev16.addweb
  • Merge request !17#3452421- Added configure link. → (Merged) created by Unnamed author
  • Pipeline finished with Success
    20 days ago
    Total: 217s
    #190652
  • Status changed to Needs review 20 days ago
  • 🇮🇳India dev16.addweb

    I have created MR, Please review.

  • Status changed to Needs work 20 days ago
  • @silvi.addweb, getting error after apply MR17

    error: patch failed: agreement.info.yml:2
    error: agreement.info.yml: patch does not apply

  • Status changed to Needs review 19 days ago
  • 🇮🇳India dev16.addweb

    Hi @tanuja-bohra, can you check once, I am not getting error,
    MR is applied cleanly.

    custom/agreement-3452421# curl https://git.drupalcode.org/project/agreement/-/merge_requests/17.diff | git apply -v
      % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                     Dload  Upload   Total   Spent    Left  Speed
    100   488    0   488    0     0    981      0 --:--:-- --:--:-- --:--:--   979
    Checking patch agreement.info.yml...
    Applied patch agreement.info.yml cleanly.
    
  • Status changed to RTBC 19 days ago
  • 🇵🇭Philippines paraderojether

    Hi

    I reviewed MR!17, confirmed it is applied cleanly and the configure link is added successfully.

    I added screenshots for reference.
    Thank you.

  • Pipeline finished with Skipped
    19 days ago
    #191909
  • First commit to issue fork.
  • Status changed to Fixed 19 days ago
  • 🇺🇸United States mradcliffe USA

    Oh, wow, thank you so much for the merge request and review. I appreciate it.

    I updated the issue summary to contain the resolution

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

Production build 0.69.0 2024