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

FIxed two bugs and added testimonial section #475

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AasthaSingh28
Copy link

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.

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.

  1. Pull this branch.
  2. Run npm install to install dependencies.
  3. Run npm test to execute the test suite.
  4. 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

Copy link

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! 😊

@AasthaSingh28
Copy link
Author

Hey @Trisha-tech please give me the write access so that I can resolve the conflicts by myself :)

@Trisha-tech
Copy link
Owner

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.

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.

  1. Pull this branch.
  2. Run npm install to install dependencies.
  3. Run npm test to execute the test suite.
  4. 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.

@Suman373 Suman373 added enhancement New feature or request level2 gssoc-ext hacktoberfest-accepted For valid PRs during hacktoberfest event labels Oct 16, 2024
@Suman373 Suman373 requested review from Trisha-tech and removed request for Trisha-tech October 16, 2024 14:24
@Suman373
Copy link
Collaborator

@AasthaSingh28 Follow the instructions given by Project admin and then we can merge your PR

@AasthaSingh28
Copy link
Author

@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
Copy link
Collaborator

@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.

@mridulsaggi
Copy link
Collaborator

@AasthaSingh28 accept the incoming changes and resolve the conflicts from your side to get pull request merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc-ext hacktoberfest-accepted For valid PRs during hacktoberfest event level2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature Request]: A Testimonial and review section
4 participants