[PR #1335] [CLOSED] Wip/fix task permissions and routes #3355

Closed
opened 2026-03-22 14:41:51 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/1335
Author: @aroige
Created: 8/22/2025
Status: Closed

Base: mainHead: wip/fix-task-permissions-and-routes


📝 Commits (10+)

  • 4e7abb3 Added tasks and prd
  • 5fa0470 docs: Update API V2 PRD and Task List
  • 567b74f Merge pull request #1 from aroige/docs-api-v2-roadmap
  • 650a52d docs: Add setup and testing instructions to AGENTS.md
  • dae85d8 docs: Add setup and testing instructions to AGENTS.md
  • b12bdf5 Merge pull request #2 from aroige/setup-agent-docs
  • de6466f Updated gitignore and added JULES.md
  • 075f6c3 Merge branch 'main' of https://github.com/aroige/vikunja
  • f33de07 The handler uses existing model methods to check for admin permissions and to perform the deletion. The new tests verify that successful deletions return a 204 status, and that non-existent projects or permission failures are handled correctly with 404 and 403 status codes, respectively.
  • 639d4e9 fix(tests): Correctly initialize config for test fixtures

📊 Changes

146 files changed (+8470 additions, -5299 deletions)

View changed files

.air.toml (+12 -0)
📝 .gitignore (+3 -0)
📝 .vscode/launch.json (+27 -1)
📝 AGENTS.md (+30 -0)
API_V2_PRD.md (+169 -0)
API_V2_TASKS.md (+323 -0)
FUTURE_REFACTOR.md (+77 -0)
JULES.md (+23 -0)
📝 devenv.lock (+8 -8)
📝 devenv.nix (+22 -1)
frontend/cypress/Dockerfile (+27 -0)
📝 frontend/cypress/docker-compose.yml (+3 -2)
📝 frontend/package.json (+3 -1)
📝 frontend/pnpm-lock.yaml (+2059 -2077)
📝 frontend/src/components/input/editor/TipTap.vue (+5 -2)
📝 frontend/src/components/misc/ApiConfig.vue (+5 -2)
📝 frontend/src/helpers/attachments.ts (+3 -1)
📝 frontend/src/helpers/auth.ts (+3 -1)
📝 frontend/src/helpers/checkAndSetApiUrl.ts (+48 -83)
📝 frontend/src/helpers/fetcher.ts (+1 -11)

...and 80 more files

📄 Description

No description provided


🔄 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/1335 **Author:** [@aroige](https://github.com/aroige) **Created:** 8/22/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `wip/fix-task-permissions-and-routes` --- ### 📝 Commits (10+) - [`4e7abb3`](https://github.com/go-vikunja/vikunja/commit/4e7abb33b2862e413e5b05ce793549091a5f3b71) Added tasks and prd - [`5fa0470`](https://github.com/go-vikunja/vikunja/commit/5fa0470c2283676f2fbe19bc9fd18c56a4eb0242) docs: Update API V2 PRD and Task List - [`567b74f`](https://github.com/go-vikunja/vikunja/commit/567b74f9ab7a8f93119c6a9e867ad34a31f556c6) Merge pull request #1 from aroige/docs-api-v2-roadmap - [`650a52d`](https://github.com/go-vikunja/vikunja/commit/650a52dfc303b4383c2cf15d77f556468354212e) docs: Add setup and testing instructions to AGENTS.md - [`dae85d8`](https://github.com/go-vikunja/vikunja/commit/dae85d8e9877daca11e1b2b473c00cb1f5fc617e) docs: Add setup and testing instructions to AGENTS.md - [`b12bdf5`](https://github.com/go-vikunja/vikunja/commit/b12bdf5ff693ffebb968caa1389649f9afbb7545) Merge pull request #2 from aroige/setup-agent-docs - [`de6466f`](https://github.com/go-vikunja/vikunja/commit/de6466f7a9d41ce12ee8e66ae034c683e12c226b) Updated gitignore and added JULES.md - [`075f6c3`](https://github.com/go-vikunja/vikunja/commit/075f6c36112cd0b21b973250b2fefc48255b480b) Merge branch 'main' of https://github.com/aroige/vikunja - [`f33de07`](https://github.com/go-vikunja/vikunja/commit/f33de07a8dba1eeaad8574a0ebd08c757b0b779a) The handler uses existing model methods to check for admin permissions and to perform the deletion. The new tests verify that successful deletions return a 204 status, and that non-existent projects or permission failures are handled correctly with 404 and 403 status codes, respectively. - [`639d4e9`](https://github.com/go-vikunja/vikunja/commit/639d4e9ceef9a4b3f1478235481695884f3e8e87) fix(tests): Correctly initialize config for test fixtures ### 📊 Changes **146 files changed** (+8470 additions, -5299 deletions) <details> <summary>View changed files</summary> ➕ `.air.toml` (+12 -0) 📝 `.gitignore` (+3 -0) 📝 `.vscode/launch.json` (+27 -1) 📝 `AGENTS.md` (+30 -0) ➕ `API_V2_PRD.md` (+169 -0) ➕ `API_V2_TASKS.md` (+323 -0) ➕ `FUTURE_REFACTOR.md` (+77 -0) ➕ `JULES.md` (+23 -0) 📝 `devenv.lock` (+8 -8) 📝 `devenv.nix` (+22 -1) ➕ `frontend/cypress/Dockerfile` (+27 -0) 📝 `frontend/cypress/docker-compose.yml` (+3 -2) 📝 `frontend/package.json` (+3 -1) 📝 `frontend/pnpm-lock.yaml` (+2059 -2077) 📝 `frontend/src/components/input/editor/TipTap.vue` (+5 -2) 📝 `frontend/src/components/misc/ApiConfig.vue` (+5 -2) 📝 `frontend/src/helpers/attachments.ts` (+3 -1) 📝 `frontend/src/helpers/auth.ts` (+3 -1) 📝 `frontend/src/helpers/checkAndSetApiUrl.ts` (+48 -83) 📝 `frontend/src/helpers/fetcher.ts` (+1 -11) _...and 80 more files_ </details> ### 📄 Description _No description provided_ --- <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-03-22 14:41:51 -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#3355