Skip to content

Show only version string for OTA status#1738

Merged
yujonglee merged 1 commit intomainfrom
ota-version-label-shortening
Nov 20, 2025
Merged

Show only version string for OTA status#1738
yujonglee merged 1 commit intomainfrom
ota-version-label-shortening

Conversation

@yujonglee
Copy link
Contributor

Display the truncated version string (e.g. "v1.2.3") instead of the verbose "You're up to date (v...)" message so the sidebar OTA menu item is concise and avoids the previous truncated/wordy display. Also remove text truncation on shared profile labels to prevent duplicate truncation styling and ensure labels render cleanly.

Display the truncated version string (e.g. "v1.2.3") instead of the verbose "You're up to date (v...)" message so the sidebar OTA menu item is concise and avoids the previous truncated/wordy display. Also remove text truncation on shared profile labels to prevent duplicate truncation styling and ensure labels render cleanly.
@netlify
Copy link

netlify bot commented Nov 20, 2025

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 91e0b96
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/691e625c4cb25800082d6da2
😎 Deploy Preview https://deploy-preview-1738--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Updates the OTA update checker UI component to display a simplified version label ("v" or "No updates available") in the noUpdate state, and removes CSS text truncation from MenuItem labels to allow text wrapping.

Changes

Cohort / File(s) Summary
UpdateChecker label text simplification
apps/desktop/src/components/main/sidebar/profile/ota/index.tsx
Modified noUpdate state label display: when no updates available, shows "v" if version exists, otherwise displays "No updates available"
MenuItem styling adjustment
apps/desktop/src/components/main/sidebar/profile/shared.tsx
Removed "truncate" CSS class from label span in MenuItem component; label text will now wrap instead of being truncated

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

  • Both changes are isolated and cosmetic (UI text simplification and CSS class removal)
  • No control flow or logic modifications
  • Straightforward visual/display updates affecting two separate components

Possibly related PRs

  • fastrepl/hyprnote#1584: Directly related OTA UI changes affecting the same UpdateChecker component and MenuItem styling modifications.
✨ 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 ota-version-label-shortening

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c1e6e45 and 91e0b96.

📒 Files selected for processing (2)
  • apps/desktop/src/components/main/sidebar/profile/ota/index.tsx (1 hunks)
  • apps/desktop/src/components/main/sidebar/profile/shared.tsx (1 hunks)

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

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

@yujonglee yujonglee merged commit 17270d1 into main Nov 20, 2025
7 of 9 checks passed
@yujonglee yujonglee deleted the ota-version-label-shortening branch November 20, 2025 00:37
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.

1 participant