Add a note about tracking to page content setting#2008
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughA new informational note has been added to the Changes
Possibly related PRs
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
classes/helpers/FrmOnSubmitHelper.php (1)
138-140: Excellent addition of informative note about tracking limitations.The added note provides valuable information to users about the behavior of the page content display feature and its implications for tracking. It correctly highlights potential issues with tracking mechanisms like Google Analytics and offers a helpful suggestion to use the "Redirect to Url" setting for tracking purposes.
Consider breaking the note into shorter sentences or bullet points for improved readability. For example:
<div class="frm_note_style"> <?php echo esc_html__( "Please note:\n" . "• The contents of the selected page will be displayed without triggering a full page load.\n" . "• Standard URL tracking mechanisms (e.g., Google Analytics) will not work as expected.\n" . "• For tracking purposes, consider using the 'Redirect to Url' setting instead.", 'formidable' ); ?> </div>This format may be easier for users to scan and understand quickly.
Fixes https://github.com/Strategy11/formidable-pro/issues/3268