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

Inbound parsing : améliorer le matching #910

Merged
merged 5 commits into from
Sep 18, 2023

Conversation

raphodn
Copy link
Contributor

@raphodn raphodn commented Sep 15, 2023

Suite à #901

Quoi ?

Répare des soucis potentiels de détection entre la v0 et la v1.

Dans notre base de données, on a actuellement des siae_encoded avec le format _1aa4, ce qui pourrait induire une v0 alors qu'ils sont en v1.

On regarde donc la dernière chaine de caractère après le _.

Et on évite que ce champ soit vide en le remplissant par le siae.slug si jamais le siae.contact_full_name est vide

Pourquoi ?

Dans la v1, on définit siae_encoded = siae.contact_full_name, mais cela peut arriver que contact_full_name soit vide.

@raphodn raphodn self-assigned this Sep 15, 2023
Copy link
Contributor

@madjid-asa madjid-asa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, tu veux pas profiter de la pr pour corriger le contact_full_name ? on pourrait par exemple prendre l'utilisateur de la structure ou je ne sais pas ^^

@raphodn
Copy link
Contributor Author

raphodn commented Sep 18, 2023

nice, tu veux pas profiter de la pr pour corriger le contact_full_name ? on pourrait par exemple prendre l'utilisateur de la structure ou je ne sais pas ^^

t'as raison, j'ai rajouté un commit - b415a67 - pour utiliser le siae.slug dans ce cas là !

@raphodn raphodn merged commit 6014e9e into master Sep 18, 2023
4 checks passed
@raphodn raphodn deleted the raphodn/inbound-parsing-uuid-matching-fixes branch September 18, 2023 14:19
@sentry-io
Copy link

sentry-io bot commented Sep 18, 2023

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ AttributeError: 'Conversation' object has no attribute 'set_sender_encoded' lemarche.conversations.migrations.0012_conversa... View Issue
  • ‼️ Conversation.MultipleObjectsReturned: get() returned more than one Conversation -- it returned 16! /api/inbound-email-parsing/ View Issue

Did you find this useful? React with a 👍 or 👎

raphodn added a commit that referenced this pull request Sep 18, 2023
raphodn added a commit that referenced this pull request Sep 19, 2023
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.

2 participants