Skip to content

Conversation

@barthc
Copy link
Contributor

@barthc barthc commented Nov 20, 2024

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2755652844/73591

Summary

When cache buster is enabled on a form, the GPMPN, the custom starting page is not applied to the form. This fix here is to use the gpmpn_default_page filter hook to pass in the custom page param.

@github-actions
Copy link

Warnings
⚠️ When ready, don't forget to request reviews on this pull request from your fellow wizards.

Generated by 🚫 dangerJS against 7dbb667

Copy link
Contributor

@claygriffiths claygriffiths left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick question, it might be possible to simplify this.

Otherwise approved.

@claygriffiths claygriffiths changed the title gw-cache-buster.php: Fixed GPMPN compatibility issue where custom starting page is not applied. gw-cache-buster.php: Fixed GP Multi-page Navigation compatibility issue where custom starting page is not applied. Dec 2, 2024
Copy link
Contributor

@claygriffiths claygriffiths left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@claygriffiths claygriffiths force-pushed the barth/fix/73591-gpmpn-custom-starting-page branch from 7dbb667 to 9fec98c Compare December 2, 2024 17:53
…ssue where custom starting pages are not applied.
@claygriffiths claygriffiths force-pushed the barth/fix/73591-gpmpn-custom-starting-page branch from 9fec98c to 052c9c0 Compare December 2, 2024 17:54
@claygriffiths claygriffiths changed the title gw-cache-buster.php: Fixed GP Multi-page Navigation compatibility issue where custom starting page is not applied. gw-cache-buster.php: Fixed GP Multi-Page Navigation compatibility issue where custom starting page is not applied. Dec 2, 2024
@claygriffiths claygriffiths merged commit f1958e2 into master Dec 2, 2024
1 of 3 checks passed
@claygriffiths claygriffiths deleted the barth/fix/73591-gpmpn-custom-starting-page branch December 2, 2024 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants