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] partner_data_vies_populator #1642

Open
wants to merge 14 commits into
base: 16.0
Choose a base branch
from

Conversation

dreispt
Copy link
Member

@dreispt dreispt commented Jan 11, 2024

Includes improvement:

  • [IMP] partner_data_vies_populator: populate Zip code and City

@dreispt dreispt force-pushed the 16.0-mig-partner_data_vies_populator branch 5 times, most recently from a837f35 to 8fbbfd0 Compare January 11, 2024 01:51
@dreispt dreispt force-pushed the 16.0-mig-partner_data_vies_populator branch 2 times, most recently from 6ffaf96 to 8d99650 Compare January 11, 2024 02:03
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 12, 2024
@dreispt
Copy link
Member Author

dreispt commented May 16, 2024

Ping!

@@ -0,0 +1,2 @@
This module require the `python-stdnum <https://pypi.org/project/python-stdnum/>`_ library.
As Odoo itself depends on this librairy, it should already be installed on your system.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
As Odoo itself depends on this librairy, it should already be installed on your system.
As Odoo itself depends on this library, it should already be installed on your system.

Copy link
Contributor

Choose a reason for hiding this comment

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

@dreispt Please attend to this?

@bosd
Copy link
Contributor

bosd commented May 16, 2024

Would like to do a func rev. Of the new feature. But runboat is not available.

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 19, 2024
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 22, 2024
@bosd
Copy link
Contributor

bosd commented Sep 23, 2024

@dreispt can you recreate a runboat?

@dreispt dreispt force-pushed the 16.0-mig-partner_data_vies_populator branch from 8d99650 to 96e9267 Compare September 23, 2024 15:42
@dreispt
Copy link
Member Author

dreispt commented Sep 23, 2024

@bosd Rebased and pushed. That should make it available on Runboat.

Copy link
Contributor

@bosd bosd left a comment

Choose a reason for hiding this comment

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

Thanks for your work on this!
Functional test:
The code for updating zip code and city name is not working for VAT numbers from the Netherlands.
Quick code review, minor grammar changes


So, when the data is retrieved, it will try to populate the *City* and *Zip* fields
from the last line of the address.
T he *Street* field will contain the remaining information.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
T he *Street* field will contain the remaining information.
The *Street* field will contain the remaining information.

@@ -0,0 +1,2 @@
This module require the `python-stdnum <https://pypi.org/project/python-stdnum/>`_ library.
As Odoo itself depends on this librairy, it should already be installed on your system.
Copy link
Contributor

Choose a reason for hiding this comment

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

@dreispt Please attend to this?

@github-actions github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Sep 29, 2024
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.

9 participants