Added testimonials section

[sarthakkundra]

Created a new folder src/views/HomeView/Testimonials having the card layout and an index.js file to map through the data in src/views/HomeViewData.js to show the testimonials. #28

Side note :- I did a last minute change to make the alternate name dynamic in case there's no picture for the avatar which caused a merge conflict. Maybe this will help in resolving it. File :- src/views/HomeView/Testimonials/TestimonialCard.js Line :- 45

[Abhishek-kumar09]

Hey @sarthakkundra ,

CI is failing because you have many unused variables...Delete unused variables and deploy will work fine:)

I think npm run format could help you in formatting and removing unused variables.

[Abhishek-kumar09]

Also, rebase your branch to current development branch to resolve merge conflicts:
Its easy... just do

git remote add upstream https://github.com/codeforcauseorg/codeforcause.org.git
git fetch upstream
git rebase upstream/development

Then you will have to resolve merge conflicts on your own... There will be two code blocks one is written by you and other is written in current development branch... You can just delete any of the two codeblocks, that works for your PR.

once done:
run

npm run start

to confirm everything is working fine.

Now do:

git push -f

and we can see your changes 🥇

[sarthakkundra]

@Abhishek-kumar09 Thanks for the help will do the changes :)

[Abhishek-kumar09]

@sarthakkundra , Is it running in your local machine, the build is failing here.
Also, you have to rebase your PR anyways to get it merged with the latest code. .Please tell me when you stuck somewhere, we are there to help you.

[sarthakkundra]

@Abhishek-kumar09 I have made the changes and I was just pushing them after rebasing but I'm getting committing is not possible because I have unmerged files after resolving the conflict issues. I'm trying to fix that

[Abhishek-kumar09]

have you done rebasing?
you have to force push the branch after rebasing...because git history changes with rebasing.

[sarthakkundra]

@Abhishek-kumar09 Done! Thanks a lot for the help! 💯

[Abhishek-kumar09]

Booyah! you have made it...:)

[Abhishek-kumar09]

@GangaChatrvedi @KeenWarrior , please review.

[Abhishek-kumar09]

Changes

[Abhishek-kumar09]

Please undo this change.

[KeenWarrior]

Kudos to your work @sarthakkundra

There are some changes.

1. Pull request created extra borders on top and bottom here.

2. HTML provided was for reference. You will need to use the design specified in the issue.

Please message here in case of any doubt in this PR. And on discord for any general concern.

[Abhishek-kumar09]

There are some changes.
Pull request created extra borders on top and bottom here.

This will be fixed by this change https://github.com/codeforcauseorg/codeforcause.org/pull/46#pullrequestreview-452765869

[sarthakkundra]

@KeenWarrior I think this PR should close this issue :)
Also @Abhishek-kumar09 once again thanks for all the help while making the PR!

[KeenWarrior]

Some more changes 😄

• Please fix top and bottom padding according to this
• Hover on the picture should also create a shadow on the box.

[Abhishek-kumar09]

congrats @sarthakkundra , your first commit merged into codeforcause.org. Good Work!