Skip to content

Conversation

@Brendonovich
Copy link
Contributor

@Brendonovich Brendonovich commented Oct 31, 2025

Summary by CodeRabbit

  • Bug Fixes
    • Updated the Organization selector visibility in the Export dialog header and "Export to" section to display only when multiple organizations are available.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 31, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The ExportDialog component modifies its Organization selector visibility logic. The selector is now hidden when only a single organization exists, requiring at least two organizations to display this UI control in both the export destination area and header section.

Changes

Cohort / File(s) Change Summary
Organization selector visibility tightening
apps/desktop/src/routes/editor/ExportDialog.tsx
Updated conditional rendering from organisations().length > 0 to organisations().length > 1 in two Show blocks, hiding the Organization picker UI when only one organization is available

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Simple threshold adjustment in visibility logic
  • Single file affected with straightforward conditional changes
  • No algorithmic or structural complexity

Poem

🐰 A rabbit hops through dialogs grand,
Where org selectors once did stand.
Now hidden when but one exists,
A cleaner UI the code insists!
Simplicity in every line,
Makes export dialogs align. ✨

✨ 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 export-dialog-organisations

📜 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 913b8a2 and 986fd88.

📒 Files selected for processing (1)
  • apps/desktop/src/routes/editor/ExportDialog.tsx (1 hunks)

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.

@Brendonovich Brendonovich merged commit e9846b8 into main Oct 31, 2025
14 of 16 checks passed
@Brendonovich Brendonovich deleted the export-dialog-organisations branch October 31, 2025 12:24
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.

2 participants