Instalacion por composer

[manuelcanepa]

Una buena practica es la instalacion en magento mediante composer. Eso facilitaria la actualizacion de las versiones y requerir de otros modulos como es el caso de spipu/html2pdf que se menciona en la descripción.

[nmarquezdrubu]

@manuelcanepa gracias, lo tendremos en cuenta :)

[mhhansen]

@nmarquezdrubu necesitan un PR para esto? avisen y lo generamos

[manuelcanepa]

De hecho, si van a aceptar PR podria meter los cambios que hice para que sea funcional.

[mhhansen]

y ya que estamos, se podria borrar el README de mageplaza (?)
https://github.com/andreani-publico/magento-2.3/blob/master/app/code/DrubuNet/Andreani/README.md

[barbazul]

Provisoriamente se puede declarar directamente en el propio composer.json del proyecto para al menos poder instalarlo.

{
    "repositories": [
        {
            "type": "package",
            "package": {
                "name": "andreani/module-andreani-magento-2.3",
                "version": "1.0.0",
                "type": "magento2-module",
                "dist": {
                    "url": "https://github.com/andreani-publico/magento-2.3/archive/c3ec160a42e8871d2e13b0040794e26701f3725f.zip",
                    "type": "zip"
                },
                "autoload": {
                    "files": [
                        "app/code/DrubuNet/Andreani/registration.php"
                    ],
                    "psr-4": {
                        "DrubuNet\\Andreani\\": "app/code/DrubuNet/Andreani/"
                    }
                },
                "require": {
                    "spipu/html2pdf": "^5.2"
                }
            }
        }
    ]
}

[barbazul]

Para que no se pierda, ya hay un PR de Enero incorporando un composer.json

#11

Si se incorpora tal cual está, ni siquiera hace falta restructurar el repositorio para que tenga la estructura estándar de un paquete de Magento y el proceso de instalación del README incluso seguiría estando vigente para quienes instalan por Copy+Paste.