Skip to content

Conversation

@ShouryaGit023
Copy link

@ShouryaGit023 ShouryaGit023 commented Sep 19, 2025

Improved Code Structure

Separate folder for css and js
Section folder which include all the html file

Type of change

  • Improvement

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

Issue #576
Screenshot 2025-09-19 120549
Screenshot 2025-09-19 120557

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@ShouryaGit023
Copy link
Author

@supriya46788 Kindly check this PR
I have made improvements in the code structure😊

@supriya46788
Copy link
Owner

@ShouryaGit023 please resolve the conflicts

@supriya46788
Copy link
Owner

@ShouryaGit023 also upload screen recording

@ShouryaGit023
Copy link
Author

@supriya46788 I solved the merge conflict
but we will always get the merge conflict if you merge other pr before mine as what i did was to change to code structure
so if it would be okay with you then try to check and merge my pr first before accepting others to prevent this merge conflict
and after merging my PR the new code structure will be available in your repo so other contributers can pull the changes and make further contribution without any problem

i am currently closing this PR and will be sending a new one along with video attached
Hope the new one gets accepted without any fail😊

@supriya46788
Copy link
Owner

@ShouryaGit023 sure, open new pr

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.

2 participants