thedevdojo/voyager

Adding timestamps and soft deletes from table builder causes error

Ouverte

#3 364 ouverte le 29 juin 2018

Voir sur GitHub
 (1 commentaire) (0 réaction) (0 personne assignée)PHP (2 652 forks)batch import
help wantedopen for suggestionstype:enhancement

Métriques du dépôt

Stars
 (11 817 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

Version information

  • Laravel: v5.6.26
  • Voyager: v1.1.3
  • PHP: 7.2.7
  • Database: SQLite3 v3.11.0

Description

When creating a new table from within the Voyager control panel, then using the "+ Add timestamps" and "+ Add soft deletes" buttons results in an error that "timestamp" is not a valid datatype. The "timestamp" type is also not selectable in the drop-down menu, and other tables use a "datetime" field instead.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Go to 'Menu > Tools > Database > Create new'
  2. Click on either the "+ Add timestamps" or "+ Add soft delete"
  3. Error message should appear in the top-right. The data types default to "time" instead.

Expected behavior

Since SQLite3 does not internally support timestamps, auto-select another default value that does work, for example "datetime" as the other generated tables are set to.

Additional context

This application is only using an SQLite3 database in the development environment.

Guide contributeur

Direction de recherche
Examinez le code du constructeur de table de Voyager pour localiser où les horodatages et les suppressions logiques sont définis. Identifiez le mappage des types de données pour différentes bases de données. Pour SQLite, modifiez le type de données par défaut de 'timestamp' à 'datetime' pour éviter l'erreur.
Stack technique
phplaravel
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
PHPLaravel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70