-
Notifications
You must be signed in to change notification settings - Fork 110
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
Adding Star rating to the feedback form #289
Conversation
🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from team will review it as soon as possible. We appreciate your support in making Dataverse better. |
✅ Deploy Preview for multiverse-dataverse ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
The design looks file but there are some minor problems that need to be resolved:
- Remove the style tag from
index.html
file and add this styling in the relevant css file that already exists. - If you haven't made any changes in
script.js
file then copy the contents from the live github repo and paste it as it is, it will save your time resolving the conflicts.
@multiverseweb I dont have the write access for the file to resolve conflicts. |
@multiverseweb I've made the changes mentioned. |
@shriyadindi yeah now it looks good, just create a new cleaner PR so I can merge it as there are many conflicts in this one |
@multiverseweb Okay. |
@multiverseweb Created a new PR here. |
Description
Adding Star rating to the feedback form
Related Issue
Fixes #283
Dataverse.-.Brave.2024-11-05.20-32-08.mp4
Type of change
Checklist