Skip to content

Conversation

@andrextor
Copy link
Collaborator

@andrextor andrextor commented May 15, 2025

Soporte versión Laravel 12

  • Tarea: PT-14104

  • Actualización librería spatie/laravel-multitenancy a versión 4
    Guia de actualización: ver..

Dependencias :

  • php 8.2
  • Upgrade the phpUnit package to version 11.5.x.
  • Upgrade orchestra/testbench to version 10.3.x. Documentación instalación
  • Upgrade friendsofphp/php-cs-fixer to version 3.75.x.
  • Removes laravel/legacy-factories

Nota:

  • Actualización config.multitenancy con el del paquete origin
  • Actualización casos de pruebas
  • Mejoras en phpUnit.xml.dist

@andrextor andrextor requested review from davidv99 and eduarguz May 15, 2025 05:28
@andrextor andrextor force-pushed the feature/upgrade-spatie-multitenancy-laralvel-12 branch from a3a4a84 to 5f1950b Compare May 15, 2025 05:31
/*
* These fields are used by tenant:artisan command to match one or more tenant.
*/
'tenant_artisan_search_fields' => [
Copy link
Contributor

Choose a reason for hiding this comment

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

yo creo que esto no sirve, no estoy seguro, porque nosotros usamos es el domain, sería bueno ver si esto si sirve, en cualquier caso, es bueno mantener paridad con el archivo original, al menos en el orden, esto va como en la línea 27 segun el archivo original

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Se re organiza el config según el original y se separan las configuraciones propias del proyecto en apartado Placetopay configs

c0c62e4

@andrextor andrextor requested a review from eduarguz May 15, 2025 15:35
@andrextor andrextor force-pushed the feature/upgrade-spatie-multitenancy-laralvel-12 branch 2 times, most recently from 66054d4 to 13cb173 Compare May 16, 2025 01:53
@andrextor andrextor force-pushed the feature/upgrade-spatie-multitenancy-laralvel-12 branch from 13cb173 to 85d4e75 Compare May 16, 2025 03:53
@andrextor andrextor force-pushed the feature/upgrade-spatie-multitenancy-laralvel-12 branch from e0624e7 to 5e65dd8 Compare May 16, 2025 20:12
@andrextor andrextor force-pushed the feature/upgrade-spatie-multitenancy-laralvel-12 branch from d993dae to 1a41f48 Compare May 16, 2025 20:42
@eduarguz eduarguz merged commit a2d4fa3 into main May 19, 2025
2 checks passed
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.

4 participants