Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: add Form Field Manager license UI to additional fields placeholder #7814

Open
wants to merge 17 commits into
base: develop
Choose a base branch
from

Conversation

JoshuaHungDinh
Copy link
Contributor

@JoshuaHungDinh JoshuaHungDinh commented Mar 18, 2025

Resolves GIVE-1011

Description

This PR updates the additional fields placeholderr UI to consider users that may have a valid license that includes form-field-manager but have not yet installed it.

Affects

Donation Form Builder -> left sidebar blocks

Visuals

Screenshot 2025-03-18 at 2 33 39 PM

Testing Instructions

  • View the form builder with FFM installed -> you should NOT see the additional fields UI.
  • View the form builder with FFM not installed -> you should see the additional fields UI.
  • View the form builder with an all_access_pass pass & FFM not installed -> you should see the conditional UI with a link directing user to download their additional fields from Givrwp.com.

Note: all_access_pass is given to users with Plus, Pro & Agency plans.

Pre-review Checklist

  • Acceptance criteria satisfied and marked in related issue
  • Relevant @unreleased tags included in DocBlocks
  • Includes unit tests
  • Reviewed by the designer (if follows a design)
  • Self Review of code and UX completed

@JoshuaHungDinh JoshuaHungDinh marked this pull request as ready for review March 19, 2025 17:07
@@ -66,6 +67,7 @@ public function storageData(int $donationFormId): array
],
'formFieldManagerData' => [
'isInstalled' => defined('GIVE_FFM_VERSION'),
'isLicensed' => (bool) Give_License::get_license_by_plugin_dirname('give-form-field-manager'),
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this returns data about the license. Do we need to check if the license is valid or anything?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point, yea we should. 55d9685

Copy link
Contributor

@jdghinson jdghinson left a comment

Choose a reason for hiding this comment

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

All good!

Copy link
Contributor

@jonwaldstein jonwaldstein left a comment

Choose a reason for hiding this comment

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

@JoshuaHungDinh looks good, nice job!

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