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

Issue #2 || Created the folder structure and fixed some minor details (not code related) #3

Merged
merged 3 commits into from
Oct 12, 2020

Conversation

Niaz-Ul-Haque
Copy link
Contributor

@Niaz-Ul-Haque Niaz-Ul-Haque commented Oct 11, 2020

I added the folder structure but also did some whitespace removal (as you can see from the code). Everything is the same just added the folder structure.

Copy link
Contributor Author

@Niaz-Ul-Haque Niaz-Ul-Haque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just cleaned up white spaces and removed/fixed whitespaces throughout entire program.

src/App.js Outdated
import logo from './logo.svg';
import './App.css';
import React from "react";
import logo from "./logo.svg";
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you have changed this logo.svg file location in this pull request (src/logo.svg → src/images/logo.svg). so, it has to update the path here also. Due to that reason build error happed. could you change this path and push again.
Capture

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OHHHHHHHHHHH, I see. That was my bad. I'll fix it right away and do a PR. Thank you!

@kelumsampath kelumsampath linked an issue Oct 12, 2020 that may be closed by this pull request
@Niaz-Ul-Haque
Copy link
Contributor Author

Done, please do check. tytyty

@kelumsampath kelumsampath added the hacktoberfest-accepted For Hacktoberfest PRs label Oct 12, 2020
@kelumsampath kelumsampath merged commit 414bd35 into kelumsampath:dev Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted For Hacktoberfest PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create Folder Structure
2 participants