Skip to content

Conversation

@sangeethailango
Copy link
Member

@sangeethailango sangeethailango commented Jan 23, 2026

Description

  • Removed product_tour field from WorkspaceUserProperties and moved to Profile.
  • Back migrated the below JSON fields to have true in all the keys:
    product_tour
    is_navigation_tour_completed
    getting_started_checklist
    tips
    explored_features

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 23, 2026

📝 Walkthrough

Walkthrough

The changes migrate the product_tour field from WorkspaceUserProperties to the Profile model across multiple database and model definition files. This includes introducing helper functions for default values, creating two migrations to handle the schema changes and data transfer, and updating both the workspace and user models accordingly.

Changes

Cohort / File(s) Summary
Workspace model changes
apps/api/plane/db/models/workspace.py
Removed get_default_product_tour() helper function and product_tour JSONField from WorkspaceUserProperties model (11 lines removed).
User model changes
apps/api/plane/db/models/user.py
Added get_default_product_tour() helper function returning a dict with product tour flags set to False, and added product_tour JSONField to Profile model with this function as the default (11 lines added).
Migration schema updates
apps/api/plane/db/migrations/0113_webhook_version.py
Introduced set_default_product_tour_to_false() function and changed the product_tour field default for WorkspaceUserProperties from get_default_product_tour to the new function (10 lines changed).
Migration data transfer
apps/api/plane/db/migrations/0118_remove_workspaceuserproperties_product_tour_and_more.py
Removed product_tour from WorkspaceUserProperties and added it to Profile. Includes helper functions (set_getting_started_checklist(), set_default_tips(), set_default_explored_features(), set_default_product_tour()) and migrate_all_the_product_tour_to_true() RunPython operation to populate defaults across Profile, WorkspaceMember models (71 lines added).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 Our product tour hops from workspace views,

To profiles now—a fresh debut!

With helper functions, all flags false and bright,

Migrations guide the data right. 🎪

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The description is incomplete. It only provides the Description section, missing required sections like Type of Change, Test Scenarios, and References. Fill in the Type of Change checkbox, add Test Scenarios describing how the migration was validated, and include relevant issue references.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: migrating product tour JSON fields with true values, matching the raw summary content.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

@makeplane
Copy link

makeplane bot commented Jan 23, 2026

Linked to Plane Work Item(s)

This comment was auto-generated by Plane

@sangeethailango sangeethailango changed the title [WEB-5990] migration: back migrate all product tour fields to set true [WEB-5990] migration: back migrate product tour JSON fields with true values Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants