Fix issue with sqlite migrations for comment_seers #503

Fusionnée
vbrandl a fusionné 1 révision(s) à partir de fix/sqlite-migrations vers master il y a 5 ans
vbrandl a commenté il y a 5 ans (Migré de github.com)

This fixes the issue in the sqlite migrations for comment_seers.

The temporary workaround was to change the id column from id INTEGER PRIMARY KEY AUTOINCREMENT to id INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT.

This change renames the table, creates a new table with the correct attributes, copies the old table over in the new one and deletes the old one.

This fixes the issue in the sqlite migrations for `comment_seers`. The temporary workaround was to change the `id` column from `id INTEGER PRIMARY KEY AUTOINCREMENT` to `id INTEGER NOT NULL PRIMARY KEY AUTOINCREMENT`. This change renames the table, creates a new table with the correct attributes, copies the old table over in the new one and deletes the old one.
elegaanz (Migré de github.com) ces changements ont été approuvés il y a 5 ans
elegaanz (Migré de github.com) laisser un commentaire

Thank you. 👍

Thank you. :+1:
codecov[bot] a commenté il y a 5 ans (Migré de github.com)

Codecov Report

Merging #503 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #503   +/-   ##
=======================================
  Coverage   26.89%   26.89%           
=======================================
  Files          65       65           
  Lines        8966     8966           
=======================================
  Hits         2411     2411           
  Misses       6555     6555
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/503?src=pr&el=h1) Report > Merging [#503](https://codecov.io/gh/Plume-org/Plume/pull/503?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/21cb0ef43786adee42065cecf053b8e525fe2c3f?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. ```diff @@ Coverage Diff @@ ## master #503 +/- ## ======================================= Coverage 26.89% 26.89% ======================================= Files 65 65 Lines 8966 8966 ======================================= Hits 2411 2411 Misses 6555 6555 ```

Relecteurs

La demande d'ajout a été fusionnée en f0d6b9d1e8.
Vous pouvez également voir les instructions en ligne de commande.

Étape 1:

Depuis le dépôt de votre projet, sélectionnez une nouvelle branche et testez les modifications.
git checkout -b fix/sqlite-migrations master
git pull origin fix/sqlite-migrations

Étape 2:

Fusionner les modifications et mettre à jour sur Forgejo.
git checkout master
git merge --no-ff fix/sqlite-migrations
git push origin master
Connectez-vous pour rejoindre cette conversation.
Aucune évaluation
Aucun jalon
Pas d'assignataires
1 participants
Notifications
Échéance
La date d’échéance est invalide ou hors plage. Veuillez utiliser le format 'aaaa-mm-dd'.

Aucune échéance n'a été définie.

Dépendances

No dependencies set.

Reference: Plume/Plume#503
Chargement…
Il n'existe pas encore de contenu.