Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resource and Support content types can not update alias #3406

Closed
3 tasks
swirtSJW opened this issue Nov 6, 2020 · 3 comments
Closed
3 tasks

Resource and Support content types can not update alias #3406

swirtSJW opened this issue Nov 6, 2020 · 3 comments
Assignees
Labels
crew-sitewide Defect Something isn't working (issue type) Drupal engineering CMS team practice area non-quarter-prio (PW) not related to quarterly priorities Public Websites Scrum team in the Sitewide crew Resources and support CMS managed product owned by Public Websites team sitewide

Comments

@swirtSJW
Copy link
Contributor

swirtSJW commented Nov 6, 2020

Notes from November 2023 (Jill):

Today, I was able to login as content admin (on Staging) and check the "Generate alias" box and save the update, so re: #2 in ticket body, that reported error validation piece of this doesn't repro anymore. I don't have a means to confirm the FE publish went through, since I was on Staging.

In general I do think we'll need to step through this report again and confirm if prod today behaves the same way or not. Could timebox this for that if we wanted to.

Description

  1. On a published R&S node: you cannot change the alias of a published node.
    If you check the box to make it use the alias pattern, it does not make it through the publishing pipeline.
    image

  2. As content-admin or admin: try to force a change in the alias field is validated / errors and will not save.
    image

  3. Bulk Operations: if you use a bulk operation to move a node or many nodes from unpublished to published, aliases do not appear to get updated.

This seems to be a collision of the following two PR's that were merged in Nov 2020:

Workaround: For now the only workaround is to use /admin/config/search/path

Repro steps

ACs

  • Test in a tugboat to attempt to reproduce.
  • If not reproducible, then close ticket.
  • If reproducible, we'll determine as a team what to do (use this ticket or create a new one)
@swirtSJW swirtSJW added Defect Something isn't working (issue type) Critical defect Typically set as "high priority" in zenhub. labels Nov 6, 2020
@swirtSJW
Copy link
Contributor Author

swirtSJW commented Nov 6, 2020

This was put up as a quick undo, but decided against.
#3408

https://dsva.slack.com/archives/CT4GZBM8F/p1604677259307200

@kevwalsh kevwalsh removed the Critical defect Typically set as "high priority" in zenhub. label Nov 9, 2020
@oksana-c oksana-c removed this from the CMS Sprint 20 - Product Support milestone Nov 24, 2020
@jilladams jilladams added the Public Websites Scrum team in the Sitewide crew label Jul 17, 2023
@wesrowe wesrowe added the non-quarter-prio (PW) not related to quarterly priorities label Sep 12, 2023
@jilladams jilladams added Drupal engineering CMS team practice area Resources and support CMS managed product owned by Public Websites team labels Nov 14, 2023
@jilladams
Copy link
Contributor

Today, I was able to login as content admin (on Staging) and check the "Generate alias" box and save the update, so re: #2 in ticket body, that reported error validation piece of this doesn't repro anymore. I don't have a means to confirm the FE publish went through, since I was on Staging.

In general I do think we'll need to step through this report again and confirm if prod today behaves the same way or not. Could timebox this for that if we wanted to.

@FranECross FranECross added the Needs refining Issue status label Dec 11, 2023
@FranECross FranECross removed the Needs refining Issue status label Jan 23, 2024
@dsasser dsasser self-assigned this Jan 31, 2024
@dsasser
Copy link
Contributor

dsasser commented Feb 9, 2024

I tested this out on Tugboat logged in as a Content Admin (not a Drupal administrator) and was not able to reproduce the issue. See below:

Screenshot 2024-02-09 at 9 08 24 AM

Per ACs, closing as a no-op.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crew-sitewide Defect Something isn't working (issue type) Drupal engineering CMS team practice area non-quarter-prio (PW) not related to quarterly priorities Public Websites Scrum team in the Sitewide crew Resources and support CMS managed product owned by Public Websites team sitewide
Projects
None yet
Development

No branches or pull requests

7 participants