Skip to content

Added Feedback section component and integrated in homepage#56

Open
skhandel826 wants to merge 1 commit intoAvishkarPatil:mainfrom
skhandel826:Feedback-Section
Open

Added Feedback section component and integrated in homepage#56
skhandel826 wants to merge 1 commit intoAvishkarPatil:mainfrom
skhandel826:Feedback-Section

Conversation

@skhandel826
Copy link

📝 Description

Brief description of changes made in this PR.

🔗 Related Issue

Fixes #(issue number)

🔄 Type of Change

  • 🐛 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 not work as expected)
  • 📚 Documentation update
  • 🎨 UI/UX improvement
  • ⚡ Performance improvement

🧪 Testing

  • I have performed a self-review of my code
  • Code has been tested locally
  • Tests pass (if applicable)
  • No new warnings introduced

📱 Frontend Changes (if applicable)

  • Responsive design tested
  • Cross-browser compatibility checked
  • Accessibility guidelines followed

🔧 Backend Changes (if applicable)

  • Database migrations created (if needed)
  • API endpoints tested
  • Authentication/authorization verified

📸 Screenshots/Demo

Add screenshots or GIFs demonstrating the changes (especially for UI changes).

📋 Additional Notes

Any additional information, dependencies, or context for reviewers.

✅ Checklist

  • My code follows the project's coding standards
  • I have updated documentation where necessary
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@AvishkarPatil
Copy link
Owner

@skhandel826 Thank you for adding the feedback section! However, the current implementation does not match the UI and design standards of our website. Please review the existing components and styling across the site to ensure consistency in layout, colors, and spacing. Also, this section appears to be a static page without any backend integration or dynamic functionality please clarify if that is intentional, or update it to follow our usual practices. Let me know if you need any reference or guidance on our standards!

@skhandel826
Copy link
Author

Ok I will review it

@AvishkarPatil
Copy link
Owner

@skhandel826 have you updated 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