Skip to content

Cleanup and screenshots#348

Closed
saurabh24thakur wants to merge 2 commits into
CCExtractor:mainfrom
saurabh24thakur:cleanup-and-screenshots
Closed

Cleanup and screenshots#348
saurabh24thakur wants to merge 2 commits into
CCExtractor:mainfrom
saurabh24thakur:cleanup-and-screenshots

Conversation

@saurabh24thakur
Copy link
Copy Markdown

Description

This PR cleans up the codebase by removing unused debug statements (console.log in frontend and fmt.Println in backend) to improve code quality and reduce noise. It also updates the README.md to ensure screenshot references are correct and verifies that the project's assets are up-to-date.

  • Fixes: # (Clean Code & Update Screenshots)

Checklist

  • Ran npx prettier --write .
  • Ran gofmt -w .
  • Ran npm test (Verified via npm run lint pre-commit hook)
  • Verified all tests pass
  • Updated documentation, if needed

Additional Notes

  • Code Cleanup: Removed console.log from HomePage.tsx, Tasks.tsx, TaskAutoSync.tsx, and other utility files. Removed fmt.Println from modify_task.go and edit_task.go.
  • Screenshots: Verified that the current screenshots in assets/ match the latest application state (hashes verified).
  • Documentation: Updated README.md to align with the current assets.

@github-actions
Copy link
Copy Markdown

Thank you for opening this PR!

Before a maintainer takes a look, it would be really helpful if you could walk through your changes using GitHub's review tools.

Please take a moment to:

  • Check the "Files changed" tab
  • Leave comments on any lines for functions, comments, etc. that are important, non-obvious, or may need attention
  • Clarify decisions you made or areas you might be unsure about and/or any future updates being considered.
  • Finally, submit all the comments!

More information on how to conduct a self review:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

This helps make the review process smoother and gives us a clearer understanding of your thought process.

Once you've added your self-review, we'll continue from our side. Thank you!

@its-me-abhishek
Copy link
Copy Markdown
Collaborator

That issue is a self assigned task for me, to be done before next release. Closing this PR for now

@saurabh24thakur
Copy link
Copy Markdown
Author

Understood, thanks for letting me know! I'll look for other unassigned issues to contribute to.

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