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

Neoletter | Form Builder | Subscription as a hidden field #542

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dawgoc
Copy link
Contributor

@dawgoc dawgoc commented Aug 21, 2023

Related to #535

To make the subscription process more straightforward, we've made a change in the Form Builder. Previously, users had to actively check a specific checkbox to subscribe. We realized this step might not be necessary for all types of subscription forms. To simplify the process, we've incorporated the subscription action as a hidden field within the form. This means that when users fill out the form, the subscription process is initiated without the need for the user to check an additional checkbox.

Currently, subscription feature-related fields are activated when the ENABLE_NEOLETTER_FORM_BUILDER_SUBSCRIPTION_FEATURE environment variable flag is set.

Without flag:
Screenshot 2023-08-21 at 14 27 35

With flag:
Screenshot 2023-08-21 at 14 13 54
Screenshot 2023-08-21 at 14 13 59
Screenshot 2023-08-21 at 14 14 10

@netlify
Copy link

netlify bot commented Aug 21, 2023

👷 Deploy request for scrivito-example pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 5692255

@apepper apepper assigned apepper and unassigned apepper Aug 31, 2023
@apepper
Copy link
Contributor

apepper commented Aug 31, 2023

FYI: I will not be able to review this PR in the near future. My queue is quite full with high priority tasks.

@jfienhold
Copy link

FYI: I will not be able to review this PR in the near future. My queue is quite full with high priority tasks.

Would be cool, if we could get this merged 😎

@apepper
Copy link
Contributor

apepper commented Oct 23, 2023

As far as I know Amier is working on an NPM package to incorporate the form builder. To me this would be the right place to add such functionality.

@aea-JR
Copy link
Contributor

aea-JR commented Dec 12, 2023

I adapted it to the Scrivito-Neoletter-Form-Widgets package and also reviewed it. The code appears to be in good shape, so perhaps you could approve it easily? Or perhaps someone else from your team could review it as well? I'm willing to take responsibility if any issues arise.

The situation is that there are existing customers who want that feature but aren't currently using the package. They've made some modifications to their form widgets that might not function properly if they switch to using the package.

@apepper
Copy link
Contributor

apepper commented Jan 3, 2024

@aea-JR : Thank you for your offer. But I think it would be useful if:

  1. The example app uses scrivito-neoletter-form-widgets directly.
  2. scrivito-neoletter-form-widgets provides a way to customers/projects to provide custom code.

Could you create a PR that removes the existing form widgets from the example app and use scrivito-neoletter-form-widgets directly?

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.

None yet

4 participants