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

Not removed parameter (geox.com) #302

Closed
serrq opened this issue Oct 18, 2023 · 2 comments · Fixed by #492
Closed

Not removed parameter (geox.com) #302

serrq opened this issue Oct 18, 2023 · 2 comments · Fixed by #492
Assignees
Labels
feature New feature or request help wanted Extra attention is needed

Comments

@serrq
Copy link

serrq commented Oct 18, 2023

Partially cleaned:

https://www.geox.com/it-IT/uomo/scarpe/stivaletti/?sfmc_activityid=a5542c58-11be-4f33-8dd5-5e0ebeae30f2

Manually cleaned:

https://www.geox.com/it-IT/uomo/scarpe/stivaletti/

1000093039

@serrq serrq changed the title Not cleaned parameter Not removed parameter Oct 18, 2023
@svenjacobs
Copy link
Owner

New sanitizers are currently blocked because I'm rewriting the internals of Leon.

@svenjacobs svenjacobs added feature New feature or request blocked Blocked by other issue labels Nov 1, 2023
@svenjacobs svenjacobs self-assigned this Nov 1, 2023
@svenjacobs svenjacobs changed the title Not removed parameter Not removed parameter (geox.com) Nov 1, 2023
@svenjacobs
Copy link
Owner

Since refactoring still takes a while, I would be happy about a PR.

@svenjacobs svenjacobs added help wanted Extra attention is needed and removed blocked Blocked by other issue labels Sep 18, 2024
jmbreuer added a commit to jmbreuer/leon that referenced this issue Oct 25, 2024
svenjacobs added a commit that referenced this issue Nov 24, 2024
* Add Salesforce sanitizer, covers #302

* chore: adjust SalesforceParametersSanitizer

---------

Co-authored-by: Sven Jacobs <github@svenjacobs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants