Changes to database selection - Saving connections
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Empieza revisando cómo almacena actualmente el plugin las credenciales de base de datos en los gráficos y, después, sigue las rutas de migración e importación/exportación. A continuación, inspecciona la UI de selección de base de datos de los gráficos y la estructura del menú de administración de WordPress. Se considera terminado cuando las conexiones son entidades independientes y reutilizables, los gráficos existentes se han migrado, la importación/exportación sigue siendo compatible y los usuarios pueden gestionar y seleccionar conexiones como se describe.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Moved from https://github.com/Codeinwp/visualizer-pro/issues/248
Context: Multiple users want to be able to re-use old connections instead of having to retype the details. We want to give them an option to re-use an older connection for that purpose.
I also thought of the scenario, if a connection has a password change, at the moment users need to modify every chart to update the change. Making connections their own entity (that can be modified one and have the changes reflected on all charts that selected it) would make sense.
Here are the changes involved for this:
1- Creation of 'database connections'
Currently the credentials are tied to a specific chart, we want to separate the chart from the connection so they become separate entities.
Connections would then be stored separately from the chart, and charts would have a "connection_id" linking to the connection.
2- Migration
Existing credentials would have to be migrated to the new format, meaning for each chart with a connection to an external database, a connection record would be created with an automated name ex : "MySQL001, MySQL002" etc. and the charts would link to it with the corresponding ID.
3- Import/Export
We have to ensure import/export will work with the new structure.
4- Manage Connections page
In the menu, between "Add new chart" and "support, we would have a "Manage Connections" page
This page would be a standard WP crud page (like feedzy's https://i.imgur.com/uBFwOop.png) and allow users to modify all details of the connection, including the name (unique) and credentials. The modification would happen on a separate page (that we can link to later). Upon submitting modifications, a confirmation message will appear: "Are you sure you want to modify this connection? This will affect all charts linked to this connection".
It would also allow creating and deleting a connection, but would not allow deleting a connection linked to a chart.
We could also display a list of charts that use this connection and upon clicking on the chart name in the last, it could open it in edit mode.
5- Update the UI to select a database
Currently we have this: https://i.imgur.com/q6DIkOt.png
Changes:
- Reword the header to "Select a database connection"
- Remove the accordion (we want to make it more obvious that the user has more options)
- Upon selecting an option that is not Wordpress DB, a second dropdown will appear containing "New Connection" as well as all existing connections for this database type.
If the user chooses an existing connection, no other fields need to be displayed. We can have a button on the right of it that says "View/Edit this connection" which would open the edit page in a new tab.
If the user chooses "New Connection", it will be same as this: https://i.imgur.com/PS0O9rQ.png
Except there will be the "New Connection" dropdown and the connection name.
The connection will be created/saved when the user saves the chart itself, if the name was left empty, it will have a generic unique name (ex: MySQL001)
- Lenguaje dominante
- PHP
- Estrellas
- 108
- Forks
- 29
- Merge medio
- 1 d 5 h
- PR fusionados (30 d)
- 8
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la 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 Codeinwp/visualizer
-
feature-request-triage
Dificultad 4/5 3-5 días Aptitud para principiantes 64/100
Codeinwp/visualizer#1388 ·
Los mantenedores suelen responder en 1 día
-
`visualizer_schedule_refresh_db` remains missing after Visualizer reactivationPosiblemente ocupada @Alexia-Soare la tomó hace 8 días. Abiertobug-report bug-report-triage customer report
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Codeinwp/visualizer#1384 · 4 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Action Scheduler queue cleanup fatals with "Unidentified action" after concurrent deletionPosiblemente ocupada @Alexia-Soare la tomó hace 11 días. Abiertobug-report bug-report-triage
Codeinwp/visualizer#1369 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Action Scheduler async queue crashes on WordPress 5.3 with `wpdb::db_server_info()` errorQuizá libre de nuevo @girishpanchal30 la tomó hace 47 días y no hay ningún pull request abierto. Abiertobug-report bug-report-triage crash-report
Codeinwp/visualizer#1361 · 5 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Add a Default value for font settingsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
Codeinwp/visualizer#1111 ·
Los mantenedores suelen responder en 1 día
Todos los issues de Codeinwp/visualizer
Issues similares
-
sync-en
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
az-digital/az_quickstart#6077 ·
Los mantenedores suelen responder en 1 día
-
Toolkit
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
[Bug] v-quick-install-app install crashes with ValueError if no supported PHP version is installedAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Add `kjkpc.net`Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
FGRibreau/mailchecker#625 ·