Fix issue with sqlite migrations for comment_seers #503

已合併

vbrandl 於將 1 次代碼提交從 fix/sqlite-migrations 合併至 master

vbrandl " 評論 " (Migrated from 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 (Migrated from github.com) approved these changes

elegaanz (Migrated from github.com) left a comment

Thank you. 👍

Thank you. :+1:

codecov[bot] " 評論 " (Migrated from 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 ```

Reviewers

The pull request has been merged as f0d6b9d1e8.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b fix/sqlite-migrations master

git pull origin fix/sqlite-migrations

Step 2:

Merge the changes and update on Forgejo.

git checkout master

git merge --no-ff fix/sqlite-migrations

git push origin master

登入才能加入這對話。

No reviewers

未選擇里程碑

No Assignees

1 參與者

訊息

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Loading…

尚未有任何內容

標籤 里程碑

Fix issue with sqlite migrations for comment_seers #503

Codecov Report

Reviewers

Step 1:

Step 2:

標籤里程碑