Update the list of Form Actions that support logging messages

[AbdiTolesa]

Fix https://github.com/Strategy11/formidable-pro/issues/6292

Summary by CodeRabbit

• Bug Fixes
  • Expanded logging coverage to include many additional third‑party integration providers (email, CRM, automation and messaging services), improving activity tracking and monitoring across more integrations.
  • No other user-facing behavior changes beyond broader log visibility.

[coderabbitai]

📝 Walkthrough

Walkthrough

The should_show_log_message function in FrmFormActionsController was updated to expand its logging whitelist array to include many additional provider action types while preserving the original return logic.

Changes

Cohort / File(s)	Summary
Logging Whitelist Expansion classes/controllers/FrmFormActionsController.php	Expanded the $logging whitelist in should_show_log_message to add multiple integration providers (e.g., aweber, campaignmonitor, convertkit, getresponse, googlespreadsheet, hubspot, mailchimp, mailpoet, n8n, twilio, zapier, etc.), retaining the same presence-check and frm_log_autoloader condition.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I hopped through lines of code today,
I added friends to logging's play,
Aweber, Zapier, Mail, and more,
Now action traces show on store,
A tiny hop — the logs hooray! 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: updating the list of Form Actions that support logging messages, which matches the core modification in the should_show_log_message function.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch issue-6292

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[deepsource-io]

Here's the code health analysis summary for commits 8f7ba69..07b87bd. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
PHP	✅ Success		View Check ↗
JavaScript	✅ Success		View Check ↗

---

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@classes/controllers/FrmFormActionsController.php`:
- Around line 504-507: The should_show_log_message function omits 'convertkit'
from the $logging list even though convertkit is registered elsewhere; confirm
whether that exclusion is intentional (e.g., premium/upgrade behavior handled in
the exclusion code path) and then either add 'convertkit' to the $logging array
in should_show_log_message to enable logging for that integration or, if
exclusion is deliberate, add a brief comment in should_show_log_message
referencing the exclusion reason and where convertkit is handled (the convertkit
registration/exclusion code) so future readers understand why it's absent;
ensure you update the $logging array within should_show_log_message (or the
comment) accordingly and keep campaignmonitor behavior unchanged.