[PR #937] [MERGED] fix(migration): detect header lines in csv file when importing from TickTick #8844

Closed
opened 2026-04-23 08:30:15 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/937
Author: @kolaente
Created: 6/13/2025
Status: Merged
Merged: 6/13/2025
Merged by: @kolaente

Base: mainHead: codex/fix-ticktick-migration-issue-and-add-tests


📝 Commits (1)

  • 44acf93 fix: improve TickTick import header detection

📊 Changes

2 files changed (+46 additions, -2 deletions)

View changed files

📝 pkg/modules/migration/ticktick/ticktick.go (+25 -2)
📝 pkg/modules/migration/ticktick/ticktick_test.go (+21 -0)

📄 Description

Might resolve https://community.vikunja.io/t/ticktick-import-most-tasks-are-ignored/3620/8

Summary

  • handle variable header lines in TickTick migrator
  • test parsing with different numbers of leading lines

Testing

  • mage lint:fix
  • mage test:unit

https://chatgpt.com/codex/tasks/task_e_6849cfd46e6c832293dd55781ff39e50


🔄 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/937 **Author:** [@kolaente](https://github.com/kolaente) **Created:** 6/13/2025 **Status:** ✅ Merged **Merged:** 6/13/2025 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `codex/fix-ticktick-migration-issue-and-add-tests` --- ### 📝 Commits (1) - [`44acf93`](https://github.com/go-vikunja/vikunja/commit/44acf9376ce5a29c6c6ea883981af0ffd7d4d66f) fix: improve TickTick import header detection ### 📊 Changes **2 files changed** (+46 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `pkg/modules/migration/ticktick/ticktick.go` (+25 -2) 📝 `pkg/modules/migration/ticktick/ticktick_test.go` (+21 -0) </details> ### 📄 Description Might resolve https://community.vikunja.io/t/ticktick-import-most-tasks-are-ignored/3620/8 ## Summary - handle variable header lines in TickTick migrator - test parsing with different numbers of leading lines ## Testing - `mage lint:fix` - `mage test:unit` ------ https://chatgpt.com/codex/tasks/task_e_6849cfd46e6c832293dd55781ff39e50 --- <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 2026-04-23 08:30:15 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#8844