Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhance In-App Notifications: Implement Dynamic Badge, Redux Integration, and Component Optimization #148

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

lokeshwar777
Copy link
Collaborator

Description

This PR implements the in-app notification functionality with several key enhancements:

  • Notification Badge in SideBar: Updated the SideBar component to include a dynamic notification count badge (commit 6270be6).
  • Redux Integration: Integrated the Notification component with Redux, improving state management and updating the UI - (commit d31ccc7).
  • Component Cleanup: Removed unused notification components and optimized styles for better performance (commit 73060d2).
  • Notification Functionality: Added core notification functionality, including actions and reducers for managing notifications within the app (commits c35c94d, 3f90d77).
  • Code Optimization: Organized and optimized import statements in the actions file to maintain clean and efficient code (commit bfd5eb1).

Related Issue

This PR is related to the enhancement request for implementing an in-app notification system with dynamic updates.
Closes #147

Motivation and Context

The changes are required to introduce a dynamic in-app notification system that integrates with Redux for better state management. This implementation replaces the temporary hard-coded notifications and lays the foundation for a scalable notification feature that enhances user engagement and experience on the platform.

How Has This Been Tested?

  • Environment: Tested on macOS with Chrome version 128.0.6613.85.
  • Functionality Testing: Verified that the notification badge updates correctly based on unread notifications. Ensured that the notifications are fetched and displayed accurately.
  • UI Testing: Confirmed that UI improvements and component cleanup did not introduce any regressions.
  • Redux Testing: Checked that the Redux integration handles the notification state effectively across different user sessions.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@ABHISHEK-PANDEY2 ABHISHEK-PANDEY2 requested review from ABHISHEK-PANDEY2, shivareddy6 and Niweera and removed request for Niweera August 31, 2024 13:41
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.

Replace Hard-Coded Notifications with Dynamic, In-App Notification System for Codelabz
3 participants