node path_alias does not create revision

I came here from Google - seems this is still the case in Drupal 10.3.10.
I found https://www.drupal.org/node/3013865 → and https://www.drupal.org/project/drupal/issues/1154180 → , but path alias still doesn't seem to show up when comparing revisions using the diff module.

I have created an MR. Path alias is now made revisionable and revisions get created for the updates made. Please check

Unassigning and submitting for review

Issue summary is incomplete. but briefly looking at the change seems you're making user a hard dependency for path_alias now and not sure that's the route we want to take

@smustgrave, I have removed some unwanted user dependencies in the tests, but this will be required for few tests as revision_uid field will be included for revisionable Path alias. Updated the issue summary and the MR. Please check.

Don’t think this is the right direction as it’s making user a dependency of path alias now

This was partially done in #3007669: Add publishing status to path aliases → but I don't think we had all the various interfaces at that point.

@smustgrave can you explain why you think the user dependency is a problem? User is a required module so it's not really making the problem worse. If we really wanted to decouple revisions from the user module we'd need to find a way to handle revision_uid differently, but I think that could be done generically in its own ticket.

Left a comment on the MR - update path doesn't look right to me.

@catch, Moving this to review as the comments in the MR are addressed. Please check.

I'm missing why so much of this is necessary, after https://www.drupal.org/node/3013865 → already landed.

This could use test coverage for the original steps to reproduce. It seems like the main issue is that new revisions aren't created by default. Also have you tried using nodes + path aliases + workspaces together?

@catch I’m eager to learn about the proper way to do this. On a site I work on, we are using a presave hook that copies the path alias into a hidden text field just so that we can track any changes to path alias in the revision tab. I’d love to know if there’s a better way to do this if you don’t mind sparing a few minute to share!