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

Split ContactsCollection #1257

Closed
chibenwa opened this issue Oct 22, 2024 · 0 comments · Fixed by #1308
Closed

Split ContactsCollection #1257

chibenwa opened this issue Oct 22, 2024 · 0 comments · Fixed by #1308
Assignees
Labels
bug Something isn't working good first issue Good for newcomers openpaas-integration

Comments

@chibenwa
Copy link
Member

Why?

Because it does both indexing of the contact locally and forwards it to OpenPaaS

However with #1163 OpenPaaS is the source of truth and would anyway forward back collected contacts to James if any

CF

image

We thus do not want given that feature to index contacts locally.

(We do not want contact auto-collection to override openpaas Data)

How

Drop ContactsCollection

Use directly James' ContactExtractor for interop with OpenPaaS instread.

For standalone James write a IndexContact mailet indexing the contact locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers openpaas-integration
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants