-
Notifications
You must be signed in to change notification settings - Fork 214
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
Add a pop up dialog box for Polls/Survey done ! #702 🌟🌟🌟 #724
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job, @Subhajit-2023-44! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.
We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines
✅ Deploy Preview for collect-your-gamingtools ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@swaraj-das @AnitSarkar123 done, please merge my PR with level 3.. I add a pop up dialog box for Polls/Survey (if you vote once, you won't see it again after reload..) #702 |
Great 👍 @Subhajit-2023-44 |
@swaraj-das Marge it |
Thank You 🙏🏻 |
Hi there! This issue is still open. We are looking forward to your response. |
@swaraj-das please merge my PR with Level 3 Thank You |
@@ -754,6 +754,210 @@ <h2>Feedback Form</h2> | |||
|
|||
</script> | |||
</div> | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've told everyone multiple times: don’t add CSS or JavaScript directly in HTML file. It will be harder to manage for index.html . When I have to take over the project after your PR, it creates unnecessary issues and makes my job harder. Follow the project structure properly. Let’s keep it clean—separate HTML, CSS, and JavaScript. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@swaraj-das okay wait I will give a new PR soon,
But you never told me 😓😓
@swaraj-das done #740 |
Hello, PR
Add new feature - #702
Add a pop up dialog box for Polls/Survey done properly -
Untitled.video.-.Made.with.Clipchamp.-.2024-10-31T105223.773.mp4
Type of PR
Checklist
Thank you for reviewing my pull request!