Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Modification de ENGINE en InnoDB de la table interactQuery en v… #1721

Merged
merged 42 commits into from
Apr 19, 2020

Conversation

lucguinchard
Copy link
Contributor

Pour tester cette PR, Il faut deux installations :

  • une fresh
  • une migration

Puis, vérifier avec la requête SQL suivante que la table est bien en InnoDB : ENGINE=InnoDB (sur la dernière ligne de la réponse) :

show create table interactQuery;

lucguinchard and others added 5 commits March 20, 2020 15:36
PHP Warning:  Illegal string offset 'class' in /usr/share/nextdom/src/Helpers/NextDomHelper.php on line 859
- Ajout du de l'entrée `widget` dans `nexdom.config.php`.
- Ajout des fichiers widgets : widgets.php Ajaxwidgets.php widgets.js widgets.scss ....widgetsManager.php.
- Ajout des templates spécifiques pour les widges.
- modification des version de Jeedom et Nextdom.
- Ajout dans le menu un bouton `widgets`.
- Ajout dans les paramètres admin un bouton `widgets`.
- Visualisation des widgets sur une page spécifique.
- Suppression du TypeEntity dans Scénario.
- Ajout de champ `cmd`.`html`.


Reste à faire :
- Stabilisation de l'affichage de la liste des widgets.
- Lors du chargement de l'édition d'un widget le select du template n'affiche pas la bonne valeur.
- Stabilisation de l'affichage de prévisualisation des widgets.
- Affectation du widget à une commande sur la page widget.
- Affectation du widget à une commande sur la page commande.
- Suppression du champs `scenario`.`type` lors d'une migration en 0.8.0
- Ajout du champs `cmd`.`html` lors d'une migration en 0.8.0
- Réalisation des tests unitaires
@Esso1-jeedom
Copy link
Contributor

fresh install
image
et aprés une restoration
image

@lucguinchard
Copy link
Contributor Author

fix #1726

@Esso1-jeedom
Copy link
Contributor

c good après la restauration d'un backup
image

@Esso1-jeedom Esso1-jeedom self-requested a review April 3, 2020 16:24
@iWils iWils removed the request for review from therealcorwin April 3, 2020 17:43
Copy link
Contributor

@iWils iWils left a comment

Choose a reason for hiding this comment

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

Test et approuvé:

  • FirstInstall + Restore backup Nextdom 0.7.0 => Ok
  • FirstInstall + Restore backup Jeedom 3.3.35 => Ok

@therealcorwin therealcorwin self-requested a review April 3, 2020 19:25
@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.

Bug A 0 Bugs
Vulnerability E 2 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 32 Code Smells

No Coverage information No Coverage information
8.0% 8.0% Duplication

@Sylvaner Sylvaner merged commit 5773555 into develop Apr 19, 2020
@lucguinchard
Copy link
Contributor Author

fix #1718

@lucguinchard lucguinchard deleted the changeInteractQueryToInnoDB branch April 23, 2020 07:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants