Skip to content

Conversation

@sambhusbabu
Copy link

Added a filter to display forms linked to selected Facebook pages.

Added a filter to display forms linked to selected Facebook pages.
Copilot AI review requested due to automatic review settings November 19, 2025 05:42
Copilot finished reviewing on behalf of sambhusbabu November 19, 2025 05:42
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a filter mechanism for Facebook lead forms to ensure only forms associated with the selected Facebook page are displayed.

Key Changes:

  • Added a facebook_page event handler that filters the facebook_lead_form field based on the selected page
  • Introduced query filtering logic to improve user experience by limiting form options to relevant entries

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@shariquerik
Copy link
Member

@sambhusbabu Add screenshots or screen recording

@shariquerik shariquerik changed the title fest: Implement filter for Facebook lead forms feat: Implement filter for Facebook lead forms Nov 19, 2025
@sambhusbabu
Copy link
Author

Screenshot 2025-11-19 at 2 08 44 PM

Add a filter to Desk view Facebook lead form

Comment on lines +12 to +13
// added a filter to show form only linked to pages
facebook_page(frm) {
Copy link
Collaborator

@NagariaHussain NagariaHussain Nov 24, 2025

Choose a reason for hiding this comment

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

Let's add an early exit if frm.doc.facebook_page is empty/falsy. What happens when we clear the Facebook page field and then input in the form field?

Copy link
Author

Choose a reason for hiding this comment

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

I will fix

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.

3 participants