Feature Enhancements#102
Conversation
|
@213sanjana is attempting to deploy a commit to the Dhanush Nehru's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This PR introduces several new features and bug fixes to enhance the note-taking application. The changes add pinned notes functionality, file attachment support, note sharing capabilities, and improvements to the PDF export feature.
Key Changes:
- Added pinned notes with dedicated UI section and pin/unpin toggle
- Implemented file attachment system using IndexedDB for storage with support for images and PDFs
- Added share functionality with options for direct link, clipboard copy, and email sharing
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/types/note.ts | Extended Note interface with pinned flag, tags, and attachment metadata |
| src/pages/Index.tsx | Added pinned notes view, URL synchronization for note sharing, and attachment handlers |
| src/lib/attachments.ts | New IndexedDB wrapper for storing and retrieving attachment blobs |
| src/hooks/useNotes.ts | Added pin toggle, attachment management functions (add/remove/get) |
| src/components/NotesSidebar.tsx | Enhanced sidebar with pinned section, improved PDF export error handling |
| src/components/NoteEditor.tsx | Added attachment UI with file picker, download, and remove functionality |
| src/components/NoteCard.tsx | Added pin button and share dropdown menu with multiple sharing options |
| README.md | Documented new attachment feature and usage guidelines |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Please check the PR @DhanushNehru |
|
@213sanjana check the copilot comments and handle accordingly |
Made the changes @DhanushNehru |
|
Please check the PR @DhanushNehru |
|
@DhanushNehru Please add the hacktoberfest- accepted label or my PR wont be considered |
DhanushNehru
added
enhancement
💡Changes Made-
🛠️ Bug Fix
-Fixed the Export as PDF issue to ensure smooth downloading without causing the page to go blank.
-Improved error handling and optimized the export flow.
✨✨ New Features Added-
✨ Share Button for Notes
-Added a “Share” button on each note card.
-Allows sharing via direct link, email, or supported platforms.
✨ Pinned Notes
-Introduced a ⭐ button to mark notes as pins.
-Added a dedicated “Pinned” section to quickly access pinned notes.
✨ Attachment Support
-Users can now attach files, images, or documents to their notes.
-Attachments are viewable and downloadable directly from the note.
Closes Issue: #98
@DhanushNehru
Hacktoberfest25