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

Moved html files to new folder #55

Closed
wants to merge 2 commits into from
Closed

Moved html files to new folder #55

wants to merge 2 commits into from

Conversation

shaurya5
Copy link

@shaurya5 shaurya5 commented Oct 4, 2021

Moved html files to pages/html/src and changed the src links in js and HTML files.

@ADI10HERO
Copy link
Member

Hey @shaurya5, I think there's some misunderstanding :(
The HTML files were to be moved to pages "or" HTML "or" src folders not pages/html/src like the literal sense of folder-tree. Like whichever you see fit.

@shaurya5
Copy link
Author

shaurya5 commented Oct 6, 2021

Hey @shaurya5, I think there's some misunderstanding :(
The HTML files were to be moved to pages "or" HTML "or" src folders not pages/html/src like the literal sense of folder-tree. Like whichever you see fit.

oh ok.. so you want every html file except index.html, in a single folder, right?

@ADI10HERO
Copy link
Member

Hey @shaurya5, I think there's some misunderstanding :(
The HTML files were to be moved to pages "or" HTML "or" src folders not pages/html/src like the literal sense of folder-tree. Like whichever you see fit.

oh ok.. so you want every html file except index.html, in a single folder, right?

Yes, that's correct.

@shaurya5
Copy link
Author

shaurya5 commented Oct 6, 2021

@ADI10HERO made a new commit. Please check

@shaurya5
Copy link
Author

shaurya5 commented Oct 7, 2021

@ADI10HERO if the commits are okay this time, please merge the change

@ADI10HERO
Copy link
Member

Hey @shaurya5, did you test your changes?

@ADI10HERO
Copy link
Member

Anyway, the PR breaks a lot of website functionality and we decided to leave the HTML files as it is for now..
Please take up other issues :) (github.com/Auto-DL)

Thank you so much for contributing! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants