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 support to the Visual Form Builder #65

Merged
merged 22 commits into from
May 24, 2024

Conversation

glaubersilva
Copy link
Contributor

@glaubersilva glaubersilva commented Mar 15, 2024

Description

This PR creates the Form Extension folder with a few samples demonstrating how to extend the Visual Form builder.

To accomplish that, the following items were added:

  • A sample settings page in the Form Builder;
  • A sample block in the Form Builder;
  • A conversion class that uses the Fields API to transform the sample block in HTML input when the Donation Form gets rendered.

Also, it adds the following new scripts to compile the form extension:

npm run dev:form-extension
npm run watch:form-extension

Affects

The Visual Form Builder

Visuals

image

image

image

image

Testing Instructions

Pull this branch and then follow these steps: https://github.com/impress-org/givewp-addon-boilerplate?tab=readme-ov-file#setup--installation

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

@glaubersilva glaubersilva self-assigned this Mar 15, 2024
@glaubersilva glaubersilva changed the title Feature: add support to form builder Feature: add support to the Visual Form Builder Apr 12, 2024
@glaubersilva glaubersilva marked this pull request as ready for review April 12, 2024 20:34
@glaubersilva glaubersilva requested a review from jonwaldstein May 9, 2024 14:42
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.

@glaubersilva thanks for the updates! just some minor tweaks but this is looking good. I would love to continue expanding this with multiples service providers for different domains 😄

.gitignore Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
src/FormExtension/FormExtensionServiceProvider.php Outdated Show resolved Hide resolved
src/FormExtension/FormExtensionServiceProvider.php Outdated Show resolved Hide resolved
src/FormExtension/FormExtensionServiceProvider.php Outdated Show resolved Hide resolved
@glaubersilva
Copy link
Contributor Author

@jonwaldstein Thank you! The next step is to implement some gateway samples. 🤑

@glaubersilva glaubersilva merged commit 1f44c14 into develop May 24, 2024
24 checks passed
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.

2 participants