Skip to content

docs: Remove the debug-only logs #1646#1854

Open
ankurkrgupta22122006-oss wants to merge 4 commits into
aryandas2911:mainfrom
ankurkrgupta22122006-oss:docs/remove-debug-only-logs
Open

docs: Remove the debug-only logs #1646#1854
ankurkrgupta22122006-oss wants to merge 4 commits into
aryandas2911:mainfrom
ankurkrgupta22122006-oss:docs/remove-debug-only-logs

Conversation

@ankurkrgupta22122006-oss

Copy link
Copy Markdown
Contributor

📌 Description
This PR addresses issue #1646 as issue described that there is some debug-only console.log are present in some files that were useful only when app was testing but now these logs are not useful in our project so, this PR removes that debug-only logs from files.

🔗 Related Issue
Closes #1646

🛠 Changes Made

  • it removed the debug only logs from files: frontend/src/pages/login.jsx and frontend/src/hooks/useTasks.js that were logging those things which were useful to know when app was testing.
  • it doesn't remove the logs that is not debug-only and which are useful yet now like logging something that depends on state values from any files.

it also explains that why we needed this section in README.
✅ Checklist

[✓ ] Code runs locally
[✓ ] Followed project structure
[ ✓]No console errors
[ ✓] Properly tested changes
[ ✓] Linked the issue

🚀 Notes for Reviewers

-This PR only did the documentation change and didn't messed the features of the app.

@ankurkrgupta22122006-oss

Copy link
Copy Markdown
Contributor Author

@aryandas2911 do i need to change something in this PR or it's okay?

@ankurkrgupta22122006-oss

Copy link
Copy Markdown
Contributor Author

@aryandas2911 please if any change required in this PR so please explain me or if not could you merge it please?

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.

[Cleanup] Remove leftover frontend debug console logs

2 participants