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

Elementos de CartaPorte #73

Merged
merged 14 commits into from
Nov 9, 2021
Merged

Elementos de CartaPorte #73

merged 14 commits into from
Nov 9, 2021

Conversation

AndreyPootMay
Copy link
Contributor

@AndreyPootMay AndreyPootMay commented Oct 26, 2021

Este pull request se dedica a revisar el ticket #70, para que se puedan añadir los Elementos de CartaPorte dentro de la librería de CfdiUtils, dado que sería útil trabajarlos como elementos.

Mi primer commit contiene los primeros elementos, conforme haga más commits iré añadiendo más asi que espero que no se cierre el pull-request, siempre siguiendo el TDD.

image

Lo que si me gustaría que sean revisados detalles como los namespaces y la dirección de las url's del SAT, ya que me estoy basando del estándar oficial de CartaPorte.


Hijos de CartaPorte (min, max)

  • Ubicaciones (1, 1)

    • Ubicacion (1, ilimitado)
      • Origen (0, 1)
      • Destino (0, 1)
      • Domicilio (0, 1)
  • Mercancias (1, 1)

    • Mercancia (1, ilimitado)
      • CantidadTransporta (0, ilimitado)
      • DetalleMercancia (0, 1)
    • AutotransporteFederal (0, 1)
      • IdentificacionVehicular (1, 1)
      • Remolques (0, 1)
        • Remolque (1, ilimitado)
    • TransporteMaritimo (0, 1)
      • Contenedor (1, ilimitado)
    • TransporteAereo (0, 1)
    • TransporteFerroviario (0, 1)
      • DerechosDePaso (0, ilimitado)
      • Carro (1, ilimitado)
        • Contenedor (0, ilimitado)
  • FiguraTransporte (0, 1)

    • Operadores (0, ilimitado)
      • Operador (1, ilimitado)
        • Domicilio (0, 1)
    • Propietario (0, ilimitado)
      • Domicilio (0, 1)
    • Arrendatario (0, ilimitado)
      • Domicilio (0, 1)
    • Notificado (0, ilimitado)

@eclipxe13 eclipxe13 marked this pull request as draft October 26, 2021 23:25
@eclipxe13
Copy link
Owner

eclipxe13 commented Oct 26, 2021

Los primeros elementos se ven muy bien!
Sería bueno poner a la descripción de el PR el árbol de elementos con su cardinalidad (si se espera que el nodo aparezca máximo 1 o N veces)
Gracias!

@eclipxe13 eclipxe13 self-requested a review October 26, 2021 23:29
@eclipxe13 eclipxe13 linked an issue Oct 26, 2021 that may be closed by this pull request
@eclipxe13 eclipxe13 marked this pull request as ready for review November 9, 2021 23:20
@eclipxe13 eclipxe13 merged commit 40e0421 into eclipxe13:master Nov 9, 2021
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.

Add "Carta porte" elements
2 participants