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.

Sign up for GitHub

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

Jump to bottom

Cleanup of the website #4

Open
yashrsharma44 opened this issue Dec 4, 2020 · 7 comments
Open

Cleanup of the website #4

yashrsharma44 opened this issue Dec 4, 2020 · 7 comments
Assignees
@codewithvk
Labels
help wanted Extra attention is needed Intermediate Requires Intermediate Level skills

Comments

@yashrsharma44
Copy link
Member

yashrsharma44 commented Dec 4, 2020 •
edited
Loading

The current structure contains a lot of files in a scattered way.
Restructure the files as follows -

  • src - contains the app.js and all the HTML/CSS files
  • algorithms - contains all the algorithms associated with the app
  • visualization - contains all the codes for visualization
  • ads - Contains all the Abstract Data Structures and Base Classes.

Or if time permits, a stretch goal can be to structure the codebase as - link
@yashrsharma44 yashrsharma44 added help wanted Extra attention is needed Intermediate Requires Intermediate Level skills labels Dec 4, 2020
@codewithvk
Copy link
Contributor

can you assign me? I am happy to work on this issue.

@yashrsharma44
Copy link
Member Author

Cool

@yashrsharma44
Copy link
Member Author

I have updated the Issue requirements with a reference to a codebase structure.

@codewithvk
Copy link
Contributor

Hey @yashrsharma44, You can see this https://bit.ly/2VNz2Pm. Can you review it?

@yashrsharma44
Copy link
Member Author

Hey @codewithvk please send in a PR for me to review the changes. The above link is not suitable for long term purpose, PR is easy to manage the changes and suggestions

@codewithvk codewithvk mentioned this issue Dec 7, 2020
Closed
@codewithvk
Copy link
Contributor

@yashrsharma44 Dear, Before do the structure we need to roadmap so we can easily do that. so I just create a roadmap according to the folder so it will help me to re-arrange the structure.

This was referenced Dec 7, 2020
Closed
Closed
yashrsharma44 pushed a commit that referenced this issue Dec 12, 2020
@codewithvk
Merge pull request #4 from codewithvk/all-contributors/add-iamrajiv
c9beaeb 
docs: add iamrajiv as a contributor
@yashrsharma44
Copy link
Member Author

Hi @codewithvk any updates regarding this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codewithvk codewithvk

Labels
help wanted Extra attention is needed Intermediate Requires Intermediate Level skills
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yashrsharma44 @codewithvk