[WEB-5523] fix: Even without edit access, the timezone is still interactive.

[b-saikrishnakanth]

Description

fix: disable timezone selection button for non-admin users
removed Universal option from timezone selection

Type of Change

• Bug fix (non-breaking change which fixes an issue)

Screenshots and Media (if applicable)

Test Scenarios

References

Summary by CodeRabbit

Changes

• Changes
  • Project timezone settings now have administrator-level access controls. Non-administrative users will find the timezone selector disabled in the project details form, preventing modifications to the timezone configuration. Administrative users retain full editing capabilities.
  • The "Universal" timezone option has been removed from the timezone selection list.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[cursor]

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on December 20.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[makeplane]

Linked to Plane Work Item(s)

• [WEB-5523] Even without edit access, the timezone is still interactive.

This comment was auto-generated by Plane

[coderabbitai]

Walkthrough

Restricts timezone editing in the project form to admin users by adding a disabled state to TimezoneSelect. Removes the Universal timezone option from available selections. No changes to validation or submission logic.

Changes

Cohort / File(s)	Summary
Timezone Access Control apps/web/core/components/project/form.tsx	Added disabled state to TimezoneSelect component, gating timezone changes to admin users only via disabled={!isAdmin}
Timezone Options apps/web/core/hooks/use-timezone.tsx	Removed Universal timezone option from the available selections list

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Straightforward conditional disabled prop addition with minimal logic
• Single-line option removal from array with no dependent logic changes
• No complex control flow or cross-file interactions to verify

Poem

🐰 A rabbit's hop through timezone gates,
Where admins now decide the fates,
No "Universal" claims to stay,
Time flows in simpler ways today! ✨⏰

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately reflects the main issue being fixed—preventing non-admin users from changing the timezone despite lacking edit access.
Description check	✅ Passed	The description addresses the main changes but lacks detail about why the Universal option was removed and is missing test scenario documentation.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-project-timezone-access

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c685042 and 2dd2f15.

📒 Files selected for processing (2)

• apps/web/core/components/project/form.tsx (1 hunks)
• apps/web/core/hooks/use-timezone.tsx (0 hunks)

💤 Files with no reviewable changes (1)

• apps/web/core/hooks/use-timezone.tsx

🧰 Additional context used

📓 Path-based instructions (1)

**/*.{ts,tsx,mts,cts}

📄 CodeRabbit inference engine (.github/instructions/typescript.instructions.md)

**/*.{ts,tsx,mts,cts}: Use const type parameters for more precise literal inference in TypeScript 5.0+
Use the satisfies operator to validate types without widening them
Leverage inferred type predicates to reduce the need for explicit is return types in filter/check functions
Use NoInfer<T> utility to block inference for specific type arguments when they should be determined by other arguments
Utilize narrowing in switch(true) blocks for control flow analysis (TypeScript 5.3+)
Rely on narrowing from direct boolean comparisons for type guards
Trust preserved narrowing in closures when variables aren't modified after the check (TypeScript 5.4+)
Use constant indices to narrow object/array properties (TypeScript 5.5+)
Use standard ECMAScript decorators (Stage 3) instead of legacy experimentalDecorators
Use using declarations for explicit resource management with Disposable pattern instead of manual cleanup (TypeScript 5.2+)
Use with { type: "json" } for import attributes; avoid deprecated assert syntax (TypeScript 5.3/5.8+)
Use import type explicitly when importing types to ensure they are erased during compilation, respecting verbatimModuleSyntax flag
Use .ts, .mts, .cts extensions in import type statements (TypeScript 5.2+)
Use import type { Type } from "mod" with { "resolution-mode": "import" } for specific module resolution contexts (TypeScript 5.3+)
Use new iterator methods (map, filter, etc.) if targeting modern environments (TypeScript 5.6+)
Utilize new Set methods like union, intersection, etc., when available (TypeScript 5.5+)
Use Object.groupBy / Map.groupBy standard methods for grouping instead of external libraries (TypeScript 5.4+)
Use Promise.withResolvers() for creating promises with exposed resolve/reject functions (TypeScript 5.7+)
Use copying array methods (toSorted, toSpliced, with) for immutable array operations (TypeScript 5.2+)
Avoid accessing instance fields via super in classes (TypeScript 5....

Files:

• apps/web/core/components/project/form.tsx

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: Build and lint web apps
• GitHub Check: Analyze (javascript)

🔇 Additional comments (1)

apps/web/core/components/project/form.tsx (1)

407-415: TimezoneSelect correctly gated behind admin-only access

Adding disabled={!isAdmin} makes the timezone picker consistent with the rest of the form controls and aligns with the PR goal of preventing non-admin edits. Assuming TimezoneSelect already supports a disabled prop, this is a straightforward and appropriate fix.

Tip

📝 Customizable high-level summaries are now available in beta!

You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.

• Provide your own instructions using the high_level_summary_instructions setting.
• Format the summary however you like (bullet lists, tables, multi-section layouts, contributor stats, etc.).
• Use high_level_summary_in_walkthrough to move the summary from the description to the walkthrough section.

Example instruction:

"Divide the high-level summary into five sections:

1. 📝 Description — Summarize the main change in 50–60 words, explaining what was done.
2. 📓 References — List relevant issues, discussions, documentation, or related PRs.
3. 📦 Dependencies & Requirements — Mention any new/updated dependencies, environment variable changes, or configuration updates.
4. 📊 Contributor Summary — Include a Markdown table showing contributions:
   | Contributor | Lines Added | Lines Removed | Files Changed |
5. ✔️ Additional Notes — Add any extra reviewer context.
   Keep each section concise (under 200 words) and use bullet or numbered lists for clarity."

Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}