chore(web): bump for cmdk Notifications empty state#718
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR updates the ChangesSubmodule Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/web`:
- Line 1: The submodule at packages/web is pinned to commit c71e2d1 which is not
an ancestor of owletto-web/main; update the submodule pointer so packages/web
references a commit that is reachable from the owletto-web/main branch (either
by rebasing/merging the upstream change into owletto-web/main or by repointing
packages/web to a valid commit SHA on owletto-web/main), then commit the updated
submodule reference so the drift check will pass.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
2a4c241 to
d31b2e4
Compare
Bumps `packages/web` to pull in lobu-ai/owletto-web#107 — the Notifications group in ⌘K is now always visible to authenticated users, with a "No notifications" placeholder when the inbox is empty.
Test plan
Summary by CodeRabbit