refactors navigation and connector APIs while improving feed/tweet filtering and SQLite date handling #93
+1,794
−692
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add unread/all toggle filter to PageFilters component with new icons - Implement inbox count aggregation for feed folders in PrimaryNavigation - Update Home page to show only unread feed items by default - Reset scroll position when navigating between pages in Layout - Simplify order toggle from dropdown menu to direct toggle button Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Auto-update read status in both detail and list views when opening a page - Refactor PageFilters to use tab-based navigation with better visual feedback - Improve filter toggle buttons with icon-only display for cleaner UI - Add optimistic cache updates for better UX responsiveness - Update README files with development status tip Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…t-dev.sh for conditional task execution
…nhance JPA configuration
Enhance the connector system with folder-based organization and filtering: - Add folder filter support to connector API and frontend navigation - Implement repository method to fetch connectors by folder ID - Update FolderConnectorView to include connector count per folder - Improve sidebar navigation with folder-based connector filtering Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Enhance the secondary sidebar with refined scrollbar appearance: - Add smooth hover transitions for scrollbar visibility - Implement subtle scrollbar that appears on hover - Wrap content sections in container for better layout control - Improve scrollbar colors and border radius for modern look Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
… API structure - Remove unused Sidebar component and its CSS module - Simplify FolderConnectorView API interface by removing duplicate folderConnectors field - Update connector-related API endpoints and services - Enhance folder filtering functionality in ConnectorController and ConnectorService - Improve page list service folder handling Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
🤖 Augment PR SummarySummary: This PR refactors navigation and connector APIs while improving feed/tweet filtering and SQLite date handling. Changes:
Technical Notes: The SQLite converter is 🤖 Was this summary useful? React with 👍 or 👎 |
![augmentcode[bot]](https://avatars.githubusercontent.com/in/1027498?s=60&v=4){kind=small}
| return ldt.atZone(ZONE_ID).toInstant(); | ||
| } catch (DateTimeParseException e2) { | ||
| // Should not happen with normal data, but handle gracefully | ||
| throw new IllegalArgumentException("Cannot parse date: " + dbData, e2); |
There was a problem hiding this comment.
convertToEntityAttribute only parses SQLite-style strings ("yyyy-MM-dd HH:mm:ss[.SSS]") and will throw for ISO-8601 values like 2026-01-11T13:30:45.123Z. That also makes InstantStringConverterTest.convertToEntityAttribute_isoFormat_returnsInstant fail and could break if any legacy rows are stored in ISO format.
🤖 Was this useful? React with 👍 or 👎
| if (event.operation !== PageOperation.delete) { | ||
| if (event.operation === PageOperation.markRead || event.operation === PageOperation.unMarkRead) { | ||
| // Handle markRead/unMarkRead operations (they don't have result) | ||
| queryClient.setQueryData<InfiniteData<PageItem[]>>(queryKey, oldData => ({ |
There was a problem hiding this comment.
In the markRead/unMarkRead branch, setQueryData assumes oldData is defined (e.g., oldData.pages), which can throw if the query isn’t in cache yet. Consider guarding against undefined here (also applies to the similar PageDetailArea markRead cache update).
🤖 Was this useful? React with 👍 or 👎
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
lcomplete
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code