✨(backend) allow to create a new user in a marketing system #1707
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
We want to create a new user in a marketing system to create a dedicated onboarding for each of them. The marketing service is implemented in the django-lasuite library and it is possible to pick the backend we want or implement a new one following the documentation on this library.
lunika
force-pushed
the
add/marketing_implem
branch
from
f6ea02c to
85700d5
Compare
AntoLC
reviewed
Dec 9, 2025
Comment on lines
+823
to
+834
| LASUITE_MARKETING = { | ||
| "BACKEND": values.Value( | ||
| "lasuite.marketing.backends.dummy.DummyBackend", | ||
| environ_name="LASUITE_MARKETING_BACKEND", | ||
| environ_prefix=None, | ||
| ), | ||
| "PARAMETERS": values.DictValue( | ||
| default={}, | ||
| environ_name="LASUITE_MARKETING_PARAMETERS", | ||
| environ_prefix=None, | ||
| ), | ||
| } |
Collaborator
There was a problem hiding this comment.
I don't see any references in the tests about LASUITE_MARKETING, should we add a doc about how it is working ?
| } | ||
|
|
||
| # Marketing and communication settings | ||
| SIGNUP_NEW_USER_TO_MARKETING_EMAIL = values.BooleanValue( |
Collaborator
There was a problem hiding this comment.
We should add them in https://github.com/suitenumerique/docs/blob/main/docs/env.md.
qbey
reviewed
Dec 9, 2025
Member
qbey
left a comment
qbey
left a comment
There was a problem hiding this comment.
Don't you need a management command to add existing users from admin?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose
We want to create a new user in a marketing system to create a dedicated
onboarding for each of them. The marketing service is implemented in the
django-lasuite library and it is possible to pick the backend we want
or implement a new one following the documentation on this library.
Proposal
Fixes #1464