Closed jams2 closed 1 month ago
Possibly related to the changes in #380
Having thought about this some more, it seems to me that it should be OK for oldUrl
to be relative - taking into account that a redirect may be "from all sites". However, it would make sense for newUrl
to always be absolute - users can either select a Page
to redirect to, which must belong to a site to be routable, or enter a URL, which validation prevents from being relative.
URLField
is really just a CharField
with validation, so it's possible that junk values exist in the database for Site.redirect_link
, but that seems unlikely under normal circumstances.
Closing this, I think the current behaviour is correct. See https://github.com/torchbox/wagtail-grapple/pull/391#discussion_r1596878747
Given the query:
and data:
I get results:
Note the new URL for
/quux
is relative.