Move the loader WP & WC version checks to the framework
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza por woocommerce/woocommerce-framework-plugin-loader-sample.php alrededor de las líneas 119-121 para entender las comprobaciones de versiones existentes de PHP, WordPress y WooCommerce. Sigue las responsabilidades del loader y de la clase base del plugin, y determina después cómo debería separarse la lógica de dependencias; se considera terminado cuando las comprobaciones ya no estén en el loader y el comportamiento del framework siga estando cubierto.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Right now our plugin loader checks for the required PHP, WordPress, and WooCommerce versions before loading the plugin itself: https://github.com/skyverge/wc-plugin-framework/blob/release-v5/woocommerce/woocommerce-framework-plugin-loader-sample.php#L119-L121
We could try and move the WP & WC version checks to the plugin base class itself to help keep the Loader class as minimal as possible. Also consider moving all of the dependency logic to its own handler class to clean things up event further.
- Lenguaje dominante
- PHP
- Estrellas
- 146
- Forks
- 43
- Merge medio
- 1 d 33 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de godaddy-wordpress/wc-plugin-framework
-
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
godaddy-wordpress/wc-plugin-framework#806 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 52/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
Google Pay Missing Environment VariablePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. AbiertoGoogle Pay JavaScript
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
-
i18n Needs Reporter Feedabck
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
godaddy-wordpress/wc-plugin-framework#536 · 1 comentario ·
Todos los issues de godaddy-wordpress/wc-plugin-framework
Issues similares
-
bug review needed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
OpenMage/magento-lts#5816 ·
Los mantenedores suelen responder en 2 días
-
priority:low type:docs
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
crazy-goat/rabbit-stream#958 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
nextcloud/richdocuments#6250 ·
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 1/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
sync-en
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día