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

[16.0][MIG] shopinvader_lead: Migration to 16.0 #1405

Closed

Conversation

matthieusaison
Copy link

@matthieusaison matthieusaison commented Sep 11, 2023

need this PR to pass test:
OCA/rest-framework#373

@matthieusaison matthieusaison force-pushed the 16.0-mig-shopinvader_api_lead branch from 69e6a6f to 1d7ab2d Compare September 11, 2023 13:27
@matthieusaison matthieusaison force-pushed the 16.0-mig-shopinvader_api_lead branch from 1d7ab2d to 0f9e1db Compare September 11, 2023 14:08
contact_name: str | None = None


# # faire un module supp pour ces champs
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO add a module: shopinvader_api_lead_firstname

else:
self.assertEqual(lead[key], check_data[key])
# TODO split shopinvader tests.test_notification
# self._check_notification("lead_confirmation", lead)
Copy link
Contributor

Choose a reason for hiding this comment

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

TODO to discuss

@sebastienbeau sebastienbeau added this to the 16.0 milestone Sep 25, 2023
@marielejeune
Copy link
Contributor

Hi @matthieusaison , @sebastienbeau
Since this PR seems to not evolve anymore, I've created a new one for shopinvader_api_lead: #1525

Mine is less complete than yours, because our customer only needs a really simple service. But the addon can then evolve if you still need it.

@sebastienbeau
Copy link
Contributor

PR have been merged here #1525

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.

3 participants