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

[BUG] #420 remove enkode_emails over entire content #435

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

Numkil
Copy link
Contributor

@Numkil Numkil commented Feb 12, 2025

Description

Removed enkode_emails on layout.twig and only add it where it makes sense fixes #420

Reason for this change

Does weird things with forms where you can edit your email afterwards like an account flow

@Numkil
Copy link
Contributor Author

Numkil commented Feb 12, 2025

@SarahOorts

Ik heb de apply blocks ook rond alle content builder blokken met text gezet.

@Numkil
Copy link
Contributor Author

Numkil commented Feb 13, 2025

Hi @SarahOorts Ik heb hier een hoop merge conflicts moeten oplossen vanwege de andere PR's daarom kan ik niet meer mergen. Zou jij deze nog een laatste keer willen reviewen en dan mergen? Merci!

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.

Remove enkode_email apply and only filter specific content.
2 participants