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

Añadir modelos para dar de baja las facturas #112

Merged
merged 11 commits into from
Oct 9, 2017

Conversation

gdalmau
Copy link
Contributor

@gdalmau gdalmau commented Sep 18, 2017

Closes #44

  • Tests
  • Serialización resource.py
  • Modelos Marshmallow
  • Service in server.py

@gdalmau gdalmau added this to the v1.5.0 milestone Sep 18, 2017
@gdalmau gdalmau self-assigned this Sep 18, 2017
@gdalmau gdalmau force-pushed the add_deregister_models branch from 2f5e216 to a0b8823 Compare September 19, 2017 09:40
@gdalmau gdalmau changed the base branch from master to developer September 19, 2017 09:46
jaumef
jaumef previously approved these changes Sep 19, 2017
@gdalmau gdalmau added blocked and removed W.I.P. labels Sep 20, 2017
@gdalmau
Copy link
Contributor Author

gdalmau commented Sep 20, 2017

Tested

Register response

OrderedDict([
    (u'CSV', 'XXXXXXXXXXXXXXXX'),
    (u'DatosPresentacion', OrderedDict([
        (u'NIFPresentador', 'XXXXXXXXX'),
        (u'TimestampPresentacion', '20-09-2017 12:59:49'),
    ])),
    (u'Cabecera', OrderedDict([
        (u'IDVersionSii', '1.0'),
        (u'Titular', OrderedDict([
            (u'NombreRazon', 'XXXXX XXXXX XXXXX'),
            (u'NIFRepresentante', None),
            (u'NIF', 'XXXXXXXXX'),
        ])),
        (u'TipoComunicacion', 'A0'),
    ])),
    (u'EstadoEnvio', 'Correcto'),  ######### Estado envío Correcto
    (u'RespuestaLinea', [
        OrderedDict([
            (u'IDFactura', OrderedDict([
                (u'IDEmisorFactura', OrderedDict([
                    (u'NIF', 'XXXXXXXXX'),
                    (u'IDOtro', None),
                ])),
                (u'NumSerieFacturaEmisor', 'FRecibOrigen-2017-09-20'),
                (u'NumSerieFacturaEmisorResumenFin', None),
                (u'FechaExpedicionFacturaEmisor', '01-12-2016'),
            ])),
            (u'EstadoRegistro', 'Correcto'),  ######### Estado registro Correcto
            (u'CodigoErrorRegistro', None),
            (u'DescripcionErrorRegistro', None),
            (u'CSV', None),
        ]),
    ]),
])

Deregister response

OrderedDict([
    (u'CSV', 'YYYYYYYYYYYYYYYY'),
    (u'DatosPresentacion', OrderedDict([
        (u'NIFPresentador', 'XXXXXXXXX'),
        (u'TimestampPresentacion', '20-09-2017 12:59:54'),
    ])),
    (u'Cabecera', OrderedDict([
        (u'IDVersionSii', '1.0'),
        (u'Titular', OrderedDict([
            (u'NombreRazon', 'XXXXX XXXXX XXXXX'),
            (u'NIFRepresentante', None),
            (u'NIF', 'XXXXXXXXX'),
        ])),
    ])),
    (u'EstadoEnvio', 'Correcto'),  ######### Estado envío Correcto
    (u'RespuestaLinea', [
        OrderedDict([
            (u'IDFactura', OrderedDict([
                (u'IDEmisorFactura', OrderedDict([
                    (u'NombreRazon', 'XXXXX XXXXX XXXXX'),
                    (u'NIF', 'XXXXXXXXX'),
                    (u'IDOtro', None),
                ])),
                (u'NumSerieFacturaEmisor', 'FRecibOrigen-2017-09-20'),
                (u'FechaExpedicionFacturaEmisor', '01-12-2016'),
            ])),
            (u'EstadoRegistro', 'Correcto'),  ######### Estado registro Correcto
            (u'CodigoErrorRegistro', None),
            (u'DescripcionErrorRegistro', None),
            (u'CSV', None),
        ]),
    ]),
])

@gdalmau gdalmau dismissed jaumef’s stale review October 9, 2017 07:50

It was a testing review, not reviewed

@gdalmau gdalmau merged commit dcd00d2 into developer Oct 9, 2017
@gdalmau gdalmau deleted the add_deregister_models branch October 9, 2017 07:52
@gdalmau gdalmau mentioned this pull request Oct 9, 2017
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.

2 participants