Skip to content

(MVR) Features master#36

Merged
igor024 merged 2 commits intomasterfrom
features-master
Mar 28, 2026
Merged

(MVR) Features master#36
igor024 merged 2 commits intomasterfrom
features-master

Conversation

@igor024
Copy link
Copy Markdown
Collaborator

@igor024 igor024 commented Mar 28, 2026

MVR version

Summary by CodeRabbit

  • New Features

    • Enhanced teleop overlay to display during active and cached time states with context-specific status messages
  • Bug Fixes

    • Updated regional event name in match schedule

@igor024 igor024 merged commit 80af3a0 into master Mar 28, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 28, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 193ee063-76c4-4219-81ee-2c5bd8659820

📥 Commits

Reviewing files that changed from the base of the PR and between b6f4870 and ddf8782.

📒 Files selected for processing (3)
  • app/src/main/java/com/team695/scoutifyapp/ui/screens/data/DataScreen.kt
  • app/src/main/java/com/team695/scoutifyapp/ui/screens/data/TeleopLockScreen.kt
  • app/src/main/java/com/team695/scoutifyapp/ui/screens/home/MatchSchedule.kt

Walkthrough

The teleop overlay composable has been refactored to accept flexible visibility and message parameters instead of a boolean flag. This allows the overlay to display both during active teleop and cached-time states with appropriate messaging. Additionally, the match event name has been updated.

Changes

Cohort / File(s) Summary
Teleop Overlay Refactoring
app/src/main/java/com/team695/scoutifyapp/ui/screens/data/TeleopLockScreen.kt, app/src/main/java/com/team695/scoutifyapp/ui/screens/data/DataScreen.kt
Updated TeleopLockScreen signature from isOverlayActive: Boolean to visible: Boolean, message: String. Visibility now includes cached-time state, and status text is now parameterized with conditional messaging for running vs. cached scenarios.
Event Name Update
app/src/main/java/com/team695/scoutifyapp/ui/screens/home/MatchSchedule.kt
Changed match event header text from "Buckeye Regional" to "Miami Valley Regional".

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 The overlay flexes with grace and care,
Parameters dance, messages in air—
Teleop running or cached in time,
Each state finds voice, each moment sublime!
Miami Valley calls, the regional name,
Scouting evolves, the adventure's aflame! 🚀

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch features-master

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.

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