github-starred/vikunja
2
0
Fork 0
mirror of https://github.com/go-vikunja/vikunja.git synced 2026-03-19 13:33:34 -05:00
Code Issues 219 Packages Projects Releases 2 Wiki Activity

[PR #1158] [MERGED] fix "null" in project views #1285

New Issue
Closed
opened 2025-11-01 21:15:02 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2025-11-01 21:15:02 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/1158
Author: @CrazyWolf13
Created: 7/22/2025
Status: ✅ Merged
Merged: 7/22/2025
Merged by: @kolaente

Base: main ← Head: patch-2

📝 Commits (5)

📊 Changes

2 files changed (+49 additions, -0 deletions)

View changed files

📝 pkg/modules/dump/restore.go (+5 -0)
📝 pkg/modules/dump/restore_test.go (+44 -0)

📄 Description

Table project_views contains NULL which was decoded to weird UTF-8 characters and compledtely broke the app.

grafik

This command fixed my DB:

sqlite3 vikunja.db "UPDATE project_views SET bucket_configuration = NULL;"

Adding this null check ensures "null" is treated as actual null.

Please again review thorougly, I'm not a go dev and that nil is only based on my research.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-vikunja/vikunja/pull/1158 **Author:** [@CrazyWolf13](https://github.com/CrazyWolf13) **Created:** 7/22/2025 **Status:** ✅ Merged **Merged:** 7/22/2025 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `patch-2` --- ### 📝 Commits (5) - [`49c8101`](https://github.com/go-vikunja/vikunja/commit/49c81010552292e86dbd643437cbfbc5715b502c) fix "null" in project views - [`deb46ed`](https://github.com/go-vikunja/vikunja/commit/deb46ede57138e8d8f7a68552208db3b0b90a59a) Update restore.go - [`4b2383e`](https://github.com/go-vikunja/vikunja/commit/4b2383ef6b6ca7514beadb33ea7e197c4a5b090f) Update restore.go - [`aacc5f5`](https://github.com/go-vikunja/vikunja/commit/aacc5f54bcc81d010d8e59bef409df97a61e9bfb) feat: add tests, make it case-insensitive - [`aad0c49`](https://github.com/go-vikunja/vikunja/commit/aad0c4931f1df211db4dec67c1bb3da1d8dad7f6) fix lint ### 📊 Changes **2 files changed** (+49 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `pkg/modules/dump/restore.go` (+5 -0) 📝 `pkg/modules/dump/restore_test.go` (+44 -0) </details> ### 📄 Description Table project_views contains NULL which was decoded to weird UTF-8 characters and compledtely broke the app. <img width="855" height="480" alt="grafik" src="https://github.com/user-attachments/assets/d3768ab6-e824-4392-9a7d-75ffe3b80265" /> This command fixed my DB: ```bash sqlite3 vikunja.db "UPDATE project_views SET bucket_configuration = NULL;" ``` Adding this null check ensures "null" is treated as actual null. Please again review thorougly, I'm not a go dev and that `nil` is only based on my research. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-01 21:15:02 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area/api
area/caldav
area/filters
area/frontend
area/gantt
area/internal-code
area/typesense
bug
changes requested
confirmed
dependencies
enhancement
good first issue
help wanted
kind/bug
kind/feature
needs reproduction
pull-request
Mirrored from GitHub Pull Request
 question
security
support
upstream issue
waiting for reply
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#1285
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?