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

Realtime Notification option on home page #88

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Shalinis19137
Copy link

@Shalinis19137 Shalinis19137 commented Oct 11, 2024

problem no. #67
📋 Description

This pull request integrates a real notification system into the research website, allowing users to receive and manage notifications about important updates, comments, and research activities in real-time. The aim is to enhance user engagement and provide timely information, thereby improving the overall user experience.

🔨 Changes Made
Implemented a real-time notification feature that fetches updates dynamically from the server.
Created a user-friendly notification interface to display notifications in a structured manner.
Added functionality to mark notifications as read and to clear all notifications.
Enhanced the UI with responsive design principles to ensure compatibility across various devices.
Updated relevant documentation to include instructions on how to use the new notification system.
✅ Checklist
Before submitting the PR, please make sure you have completed the following:

I have followed all the guidelines mentioned in CONTRIBUTING.md.
My code follows the style guidelines of this project.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have made corresponding changes to the documentation (if applicable).
I have tested it locally and it works fine.
Any dependent changes have been merged and published in downstream modules.

🏷️ Types of Changes
What type of changes does your code introduce? (Check all that apply)

Bug fix (non-breaking change which fixes an issue) 🐛

New feature (non-breaking change which adds functionality) ✨

UI enhancement (non-breaking change which enhances UI) 🎨

Documentation update 📚

📸 Screenshots
Screenshot 2024-10-11 152440

I am a beginner in this field if you like my work please provide points under gssocs-ext 2024

@@ -0,0 +1,32 @@
// script.js

// Sample notification data
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All the changes made is possibly good but the notification data(message) are fetched from the node server, we will add this functionality, just remove the sample notification data we will change the logic of displayNotification function from the API endpoint. But yah if you want to change this logic to fetch the the notifications from our node server you can work on it

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.

2 participants