FIxed two bugs and added testimonial section
Description
A clear and concise description of what the PR does.
- This PR does the following:
- Adds ... Added Testimonial.jsx and testimonial.css
- Fixes ... Fixed bugs in Home.jsx and the duplicate index.js
- Updates ... Changed the name of Index.js inside Component to index2.js
Related Issues
Link any related issues using the format Fixes #issue_number.
This helps to automatically close related issues when the PR is merged.
- Placeholder: "Fixes #417 "
Changes
List the detailed changes made in this PR.
- Added a new feature to ... Home.jsx
- Refactored the ...
- Fixed a bug in ...Fixed bugs in Home.jsx and the duplicate index.js
Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.
- Pull this branch.
- Run
npm installto install dependencies. - Run
npm testto execute the test suite. - Verify that ...
Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.
Additional Context
Any additional context or information that reviewers should be aware of.
- This PR is based on the following...
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.
- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] My changes generate no new warnings
- [ ] I am working on this issue under GSSOC
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊
Hey @Trisha-tech please give me the write access so that I can resolve the conflicts by myself :)
Description
A clear and concise description of what the PR does.
This PR does the following:
- Adds ... Added Testimonial.jsx and testimonial.css
- Fixes ... Fixed bugs in Home.jsx and the duplicate index.js
- Updates ... Changed the name of Index.js inside Component to index2.js
Related Issues
Link any related issues using the format
Fixes #issue_number. This helps to automatically close related issues when the PR is merged.
- Placeholder: "Fixes [Feature Request]: A Testimonial and review section #417 "
Changes
List the detailed changes made in this PR.
- Added a new feature to ... Home.jsx
- Refactored the ...
- Fixed a bug in ...Fixed bugs in Home.jsx and the duplicate index.js
Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.
- Pull this branch.
- Run
npm installto install dependencies.- Run
npm testto execute the test suite.- Verify that ...
Screenshots (if applicable)
Add any screenshots that help explain or visualize the changes.
Additional Context
Any additional context or information that reviewers should be aware of.
- This PR is based on the following...
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.
- [ ] I have performed a self-review of my code
- [ ] I have commented my code, particularly in hard-to-understand areas
- [ ] My changes generate no new warnings
- [ ] I am working on this issue under GSSOC
Kindly add some screenshot or screen recording of the features you have made. @AasthaSingh28 and for resolving the conflicts just pull the upstream code and accept the incoming changes.
@AasthaSingh28 Follow the instructions given by Project admin and then we can merge your PR
@Suman373 Can you please give me the write acces so that I can resolve the conflicts in this branch and make PR ready to merge asap?
@Suman373 Can you please give me the write acces so that I can resolve the conflicts in this branch and make PR ready to merge asap?
You can fix the conflicts by accepting incoming changes from upstream. I am not allowed/eligible to provide write access to contributors.
@AasthaSingh28 accept the incoming changes and resolve the conflicts from your side to get pull request merged