Add method to LP model to overwrite generated slugs if set

[web-dev-trev]

So in our twill updates the HandleSlugs trait added auto generation from title as the default behavior. This breaks any custom slugs we set like 'Articles & Videos' because it then gets set to '/article-videos'. Without this method set in the model it'll always default to it. This is a fix just for LP's but we could also set this as default behavior in the HandleSlugs trait.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[zachgarwood]

I'm a little confused by this. Where will this be called? And if the issue is the HandleSlugs trait, shouldn't we override the beforeSaveHandleSlugs() method in the LandingPageRepository?

[nikhiltri]

@web-dev-trev Can you provide clarity on this PR? I’m not seeing a reference to shouldGenerateSlugOnSave anywhere in our code or Twill’s code, so I'm not sure where this value get plugged into.

[web-dev-trev]

@nikhiltri @zachgarwood I'm not sure where this gets resolved at, I imagine there's a pattern similar to how scope attributes work that twill is looking at for this shouldGenerate. In any case I can close this and resolve it in the module repository rather than on the model.