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

Main #219

Closed
wants to merge 3 commits into from
Closed

Main #219

wants to merge 3 commits into from

Conversation

snehas-05
Copy link

Issue number #405
This PR introduces a 5-star rating and review system to the Dataverse platform, allowing users to provide feedback and rate the platform. The system is designed to engage the community, encourage quality feedback, and help potential users assess the platform's performance based on user ratings.

Key Features:
Rating System: Users can rate Dataverse from 1 to 5 stars using a user-friendly interface.

Review Submission: Alongside the rating, users can submit a detailed review describing their experience with the platform.

Real-Time Feedback Display: Submitted ratings and reviews are stored in a MySQL database and displayed immediately on the page, showing the average rating and the most recent reviews.

Data Storage: Reviews are stored in a MySQL database, including user information (name, email), the rating score, review text, and a timestamp.

Frontend: The user interface includes a responsive form for rating and review submission, as well as a section that displays the most recent reviews. The UI has been styled using custom CSS for a clean and consistent look.

Type of change

  • Bug fix
  • [ ✔] New feature

Checklist

  • [✔ ] My code follows the style guidelines of this project
  • [✔ ] I have performed a self-review of my own code
  • [✔ ] I have commented on my code, particularly in hard-to-understand areas
  • [✔ ] I have added tests that prove my fix is effective or that my feature works
  • [✔ ] New and existing unit tests pass locally with my changes
  • [✔ ] Any dependent changes have been merged and published in downstream modules

Copy link

netlify bot commented Oct 22, 2024

Deploy Preview for multiverse-dataverse ready!

Name Link
🔨 Latest commit 65df151
🔍 Latest deploy log https://app.netlify.com/sites/multiverse-dataverse/deploys/671fcc412b11e4000812740d
😎 Deploy Preview https://deploy-preview-219--multiverse-dataverse.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from Dataverse team will review it as soon as possible. We appreciate your support in making this project better

@jency1
Copy link
Collaborator

jency1 commented Oct 24, 2024

@snehas-05 Attach relevant screenshots.

@multiverseweb
Copy link
Owner

Hello @snehas-05, looks like you have created a separate page for the Star review system. Can you integrate it in the existing form:
image

as discussed in the issue #167?

Also, there is some problem with the Submit Review button, it triggers error in js:
image

Let me know if I'm doing it wrong. Hope you can fix the stated changes : )

@snehas-05
Copy link
Author

Hey @Tejas-Gupta-7 I have made some changes and committed them. Can you have a look at them for once?

@multiverseweb
Copy link
Owner

Hey @snehas-05, a new form has been created in this PR which appears at the top of hop page.
image

I was expecting a new field like this being added to the existing form. The feedback is being stored on netlify. You just need to add some radio buttons as stars that will work as rating to the existing form:

image

@multiverseweb
Copy link
Owner

You can create a new PR when you solve the issue, I'm closing this one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants