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

IMP l10n_it_fatturapa_out check on invoice lines #558

Merged
merged 1 commit into from
Jul 24, 2018

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Jul 23, 2018

ADD preventive_checks to be used for generic data tests

@eLBati eLBati added this to the 10.0 milestone Jul 23, 2018
for line in self.invoice_line_ids:
if '\n' in line.name:
raise UserError(_(
"Invoice line [%s] must not contain new line charcter"
Copy link
Member

Choose a reason for hiding this comment

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

s/charcter/character

Copy link
Member Author

Choose a reason for hiding this comment

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

@tafaRU thanks , done

@eLBati eLBati force-pushed the 10.0-imp-fatturapa-check branch from f665523 to 9d7d396 Compare July 23, 2018 09:10
@SimoRubi
Copy link
Member

@eLBati what is the issue you are experiencing with the newline character?

@eLBati
Copy link
Member Author

eLBati commented Jul 23, 2018

@SimoRubi

Type {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}String1000LatinType cannot be created from {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}String1000LatinType: Mouse
Optical

Copy link
Member

@SimoRubi SimoRubi left a comment

Choose a reason for hiding this comment

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

It might be useful to include the error you are getting in the commit, for future understanding of the change.
Anyway it's good already for me

@eLBati eLBati force-pushed the 10.0-imp-fatturapa-check branch from 9d7d396 to 932f712 Compare July 23, 2018 12:59
@eLBati
Copy link
Member Author

eLBati commented Jul 23, 2018

@SimoRubi commit message improved

@tafaRU
Copy link
Member

tafaRU commented Jul 23, 2018

@eLBati, please update the module version accordingly, then I'll merge it. Thanks!

@eLBati eLBati force-pushed the 10.0-imp-fatturapa-check branch from 932f712 to c9b89a6 Compare July 24, 2018 06:19
@eLBati
Copy link
Member Author

eLBati commented Jul 24, 2018

@tafaRU done

@tafaRU tafaRU merged commit c3dd207 into OCA:10.0 Jul 24, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Nov 16, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Dec 5, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Dec 7, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Dec 7, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Dec 7, 2018
eLBati added a commit to eLBati/l10n-italy that referenced this pull request Dec 7, 2018
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Aug 7, 2019
IMP l10n_it_fatturapa_out check on invoice lines. Otherwise user would get (OCA#558)

'Type {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}String1000LatinType cannot be created from {http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}String1000LatinType: Mouse
Optical'
ADD preventive_checks to be used for generic data tests

IMP l10n_it_fatturapa_out: showing the XML field that is failing (for ProgressivoInvio) (OCA#571)
SimoRubi pushed a commit to SimoRubi/l10n-italy that referenced this pull request Aug 7, 2019
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 6, 2020
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Dec 15, 2020
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Feb 24, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Feb 25, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Apr 2, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Apr 30, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request May 7, 2021
eLBati added a commit to TheMule71/l10n-italy that referenced this pull request May 21, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request May 26, 2021
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 2, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 2, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 10, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 10, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 11, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 11, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 25, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 25, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Nov 25, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Dec 15, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Dec 15, 2022
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Dec 30, 2022
TonyMasciI pushed a commit to saydigital/l10n-italy that referenced this pull request Jan 9, 2023
TonyMasciI pushed a commit to saydigital/l10n-italy that referenced this pull request Jan 16, 2023
TonyMasciI pushed a commit to saydigital/l10n-italy that referenced this pull request Jan 27, 2023
TheMule71 pushed a commit to TheMule71/l10n-italy that referenced this pull request Mar 9, 2023
are-agilebg pushed a commit to are-agilebg/l10n-italy that referenced this pull request May 10, 2023
are-agilebg pushed a commit to are-agilebg/l10n-italy that referenced this pull request May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants