Burn up chart start #3
Open
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
- Implement saveToLocalStorage to persist board state - Add loadFromLocalStorage to restore board on page load - Add auto-save for column/document creation, deletion, and updates - Maintain colors, positions, and content across page reloads
…ffecting project content
…d-styling Feature/fix delete and styling known issues - columns not deleting after moved causing duplicates
…details Feature/edit project details
- Improve header and navigation visibility - Add DevSync logo - Update color scheme and typography - Enhance responsive design
… functionality and improve form styling
…t list right with scrollable functionality
…ming-app Updating index and naming app
…g closer to top" This reverts commit 44f40e1.
…ming-app Update landing page spacing: adjust logo and title positioning closer…
…aming-app changed logo color, nav bar, and minor design updates
…n:id; I created Kanban_template;profile - i updated to reflect that it doesn't send you directly to the kanban; project_template was rewritten.
I added the project landing page.
Finish notifications
Add username to chat
- Integrated Socket.IO into addUserModal.js, chat.js, modal.js, profile.js, and server.js for real-time communication. - Enhanced profileRoutes to support Socket.IO and user notifications. - Added functionality to handle user joining rooms and receiving notifications in profile.js. - Updated notification rendering in notifications.ejs and profile.ejs to accommodate new real-time features. - Fixed minor bugs and improved error handling in various scripts.
…r handling; update CSS for button margins; enhance notifications.ejs for better user feedback; streamline project_template.ejs forms. Notifications update dynamically through a real-time event-driven system
Add click-to-progress functionality to Kanban documents and fix icon …
Socket notifications
…proved maintainability and organization. Removed inline script from kanban_template.ejs and updated socket handling, document management, and local storage operations. Enhanced error handling and added console logs for better debugging. BECAUSE I WAS GETTING HORRIBLE SILENT ERRORS
…ller; refactor Document and Kanban models for improved status handling almost perfect dynamic status maps.
…tatus handling in setStateList and handleProgressClick functions; ensure progress button reflects status correctly.
…ss level determination in getProject method; enhance error handling for unauthorized access.
…o getAccessLevel method; enhance error handling and logging. Update project rendering logic in getProject method. Improve CSS styles for chat layout and button display. Add event listener checks in addUserModal and projectTemplate scripts for better robustness.
More column statuses
User vs admin interface project page
… into EJS, CSS, and JS files for better organization - Move styles to dedicated kanban.css - Create separate kanban.ejs partial - Update kanban.js with modular structure - Maintain all existing functionality
…on to Document model - Implement robust progress button functionality - Update status handling for drag and drop operations - Fix status mapping and column reordering
…ristopher Ament and James Jahner as lead developers - Update project status to development in progress - Add roadmap for live demo deployment - Improve professional presentation for job applications
- Add 100+ active users badge and performance metrics - Highlight 40% latency reduction through Socket.IO optimization - Showcase Google OAuth authentication implementation - Emphasize real-time chat system and production deployment - Add project impact section demonstrating full-stack expertise - Update roadmap to reflect completed achievements - Optimize for job application presentation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| } catch (err) { | ||
| console.error(err); | ||
| req.status(500).json({ error: "Server Error" }); | ||
| } |
There was a problem hiding this comment.
Bug: Incorrect Error Handling in Notification Function
The addNotification function's error handling uses req.status(500) instead of res.status(500). The request object lacks a status method, preventing a proper server error response.
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.
No description provided.