Skip to content

fix: [bug]: network (project visibility) is silently ignored on create/upda... - #9656

Open
Roshan931 wants to merge 1 commit into
makeplane:previewfrom
Roshan931:prilog/fix-error-network-missing-from-public-api-project-08-20-2026-3e85f3
Open

fix: [bug]: network (project visibility) is silently ignored on create/upda...#9656
Roshan931 wants to merge 1 commit into
makeplane:previewfrom
Roshan931:prilog/fix-error-network-missing-from-public-api-project-08-20-2026-3e85f3

Conversation

@Roshan931

@Roshan931 Roshan931 commented Aug 20, 2026

Copy link
Copy Markdown

Fixes #9576

Root cause

network missing from public API project serializer field list

Changes

  • Added network to ProjectCreateSerializer.Meta.fields (inherited by ProjectUpdateSerializer), so POST/PATCH on /api/v1/workspaces/{slug}/projects/ now accept and persist the field; DRF's generated ChoiceField validates against Project.NETWORK_CHOICES (0/2) and rejects invalid values with a 400.

Summary by CodeRabbit

  • New Features
    • Project creation now supports specifying network details.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: cebad5a7-a171-4254-81b0-982304f6b966

📥 Commits

Reviewing files that changed from the base of the PR and between e056bbf and 8e0fc88.

📒 Files selected for processing (1)
  • apps/api/plane/api/serializers/project.py

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


📝 Walkthrough

Walkthrough

The project creation serializer now includes the network field in its accepted and serialized fields.

Changes

Project network field

Layer / File(s) Summary
Project creation serializer contract
apps/api/plane/api/serializers/project.py
ProjectCreateSerializer.Meta.fields now includes network.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to 8e0fc

This localized change enables project visibility settings to be accepted and persisted during create and update requests, with invalid values rejected by normal validation; no actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: dheeru0198, pablohashescobar

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The serializer change directly addresses issue #9576 by enabling network on project creation and updates with standard validation.
Out of Scope Changes check ✅ Passed The change is limited to adding network to the project creation serializer fields and aligns with issue #9576.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title clearly identifies the main fix: accepting project visibility through the network field during create and update operations.
Description check ✅ Passed The description explains the issue, root cause, code change, affected API operations, and validation behavior, but omits the template headings and test scenarios.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug]: network (project visibility) is silently ignored on create/update via public REST API

1 participant