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

Craft 4 #54

Open
wants to merge 67 commits into
base: main
Choose a base branch
from
Open

Craft 4 #54

wants to merge 67 commits into from

Conversation

Mosnar
Copy link
Contributor

@Mosnar Mosnar commented Nov 12, 2022

Venveo Redirect v4

Running Changelog

Added

  • Add support for new unified element editor experience
  • Add warning for when a source URI matches a URL on the site
  • Add warning for when a source URI matches an existing redirect
  • Added groups to help organize redirects
  • Added flag to track if redirects were created automatically or not
  • Added logic to delete 404's for new static redirects

Changed

  • Redirect source and destination URLs now get normalized when set on the element rather than on-save.
  • Removed element deletion redirect pruning

Fixed

  • Fixed error caused by permissions & use new element permissions interface
  • Fixed ignored 404s getting pruned if not hit for a while
  • When a redirect is created for a changed slug, the original URI is still stored in addition to the element ID.
  • Redirects that are matched that don't have a valid destination now 404 properly.
  • "New Redirect" button now always opens in a slide out

To do

  • Fix unified element editor weirdness
  • Ensure all permissions function as expected
  • Fix drafts not auto-saving when changing groups
  • Find suitable replacement for the cp meta hook to fix the "active redirects" entry slide out
  • Ensure Feed Me integration still works
  • Fix "New redirect" button
  • Merge in fixes from 3.0.11
  • Merge in fixes from 3.0.12

@Mosnar Mosnar marked this pull request as draft November 12, 2022 21:55
@Mosnar Mosnar added this to the 4.0.0 milestone Nov 13, 2022
@Mosnar Mosnar marked this pull request as ready for review January 5, 2024 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants