Removed extensive logging and comments, updated readme, removed unused

functions

Signed-off-by: Vedansh Saini <[email protected]>

Author: vedansh-5

README.md

@@ -32,6 +32,18 @@
 - Ensure the Scrum Helper settings are applied (follow step 6 above)
 -  The extension will prefill scrum content for you to edit
 
+### New Features
+1. **Standalone Popup Interface**
+   - Generate reports directly from the extension popup
+   - Live preview of the report before sending
+   - Rich text formatting with clickable links
+   - Copy report to clipboard with proper formatting
+
+### Usage Standalone
+- Click on `GENERATE` button to generate the scrum preview.
+- Edit it in the window.
+- Copy the rich HTML using the `COPY` button.
+
 ## Setting up the code locally
 
 ```
@@ -44,6 +56,8 @@ $ npm install
 
 ![POPUP](/docs/images/popup.png)
 
+![STANDALONE](docs/images/standalone.png)
+
 ## About contributing
 - Follow the Issues and PRs templates as far as possible.
 - If you want to make a PR, please mention in the corresponding issue that you are working on it.

src/scripts/popup.js

@@ -6,18 +6,15 @@ document.addEventListener('DOMContentLoaded', function() {
         this.innerHTML = '<i class="fa fa-spinner fa-spin"></i> Generating...';
         this.disabled = true;
 
-        // Call the scrum generation function
         window.generateScrumReport();
     });
 
     copyBtn.addEventListener('click', function() {
         const scrumReport = document.getElementById('scrumReport');
 
-        // Create a temporary div to manipulate the content
         const tempDiv = document.createElement('div');
         tempDiv.innerHTML = scrumReport.innerHTML;
 
-        // Convert all links to markdown format
         const links = tempDiv.getElementsByTagName('a');
         Array.from(links).forEach(link => {
             const title = link.textContent;
@@ -26,33 +23,23 @@ document.addEventListener('DOMContentLoaded', function() {
             link.outerHTML = markdownLink;
         });
 
-        // Remove the state buttons (open/closed labels)
         const stateButtons = tempDiv.getElementsByClassName('State');
         Array.from(stateButtons).forEach(button => {
             button.remove();
         });
 
-        // Replace <br> with newlines
         tempDiv.innerHTML = tempDiv.innerHTML.replace(/<br\s*\/?>/gi, '\n');
 
-        // Replace list items with proper formatting
         const listItems = tempDiv.getElementsByTagName('li');
         Array.from(listItems).forEach(item => {
-            // Add a newline before each list item and indent with a dash
             item.innerHTML = '\n- ' + item.innerHTML;
         });
 
-        // Replace <ul> and </ul> with newlines
         tempDiv.innerHTML = tempDiv.innerHTML.replace(/<\/?ul>/gi, '\n');
-        
-        // Get the text content
         let textContent = tempDiv.textContent;
+        textContent = textContent.replace(/\n\s*\n/g, '\n\n');
+        textContent = textContent.trim();
 
-        // Clean up multiple newlines and spaces
-        textContent = textContent.replace(/\n\s*\n/g, '\n\n');  // Replace multiple newlines with double newlines
-        textContent = textContent.trim();  // Remove leading/trailing whitespace
-        
-        // Copy to clipboard
         const textarea = document.createElement('textarea');
         textarea.value = textContent;
         document.body.appendChild(textarea);

src/scripts/scrumHelper.js

@@ -148,15 +148,13 @@ function allIncluded(outputTarget = 'email') {
 	}
 	// fetch github data
 	function fetchGithubData() {
-		console.log("Starting GitHub data fetch...");
 		var issueUrl = 'https://api.github.com/search/issues?q=author%3A' +
 			githubUsername +
 			'+org%3Afossasia+created%3A' +
 			startingDate +
 			'..' +
 			endingDate +
 			'&per_page=100';
-		console.log("Fetching issues from:", issueUrl);
 
 		$.ajax({
 			dataType: 'json',
@@ -170,7 +168,6 @@ function allIncluded(outputTarget = 'email') {
 				});
 			},
 			success: (data) => {
-				console.log("Received GitHub issues data:", data);
 				githubIssuesData = data;
 				writeGithubIssuesPrs();
 			},
@@ -184,7 +181,6 @@ function allIncluded(outputTarget = 'email') {
 			'..' +
 			endingDate +
 			'&per_page=100';
-		console.log("Fetching PR reviews from:", prUrl);
 
 		$.ajax({
 			dataType: 'json',
@@ -198,7 +194,6 @@ function allIncluded(outputTarget = 'email') {
 				});
 			},
 			success: (data) => {
-				console.log("Received PR reviews data:", data);
 				githubPrsReviewData = data;
 				writeGithubPrsReviews();
 			},
@@ -226,11 +221,9 @@ function allIncluded(outputTarget = 'email') {
 
 	//load initial text in scrum body
 	function writeScrumBody() {
-		console.log("writeScrumBody called");
 		if (!enableToggle) return;
 
 		setTimeout(() => {
-			console.log("generating content");
 			// Generate content first
 			var lastWeekUl = '<ul>';
 			var i;
@@ -264,7 +257,6 @@ ${userReason}`;
         }
 
 			if (outputTarget === 'popup') {
-				console.log("trying to update popup textarea");
 				const scrumReport = document.getElementById('scrumReport');
 				if (scrumReport) {
 					console.log("found div, updating content");
@@ -279,7 +271,6 @@ ${userReason}`;
 				}
 			} else {
 
-				// Use the adapter to inject content
 				const elements = window.emailClientAdapter.getEditorElements();
 				if (!elements || !elements.body) {
 					console.error('Email client editor not found');
@@ -300,11 +291,9 @@ ${userReason}`;
 		else if (projectUrl === 'open-event') project = 'Open Event';
 		return project;
 	}
-	//load initial scrum subject
 	function scrumSubjectLoaded() {
 		if (!enableToggle) return;
 		setTimeout(() => {
-			//to apply this after google has autofilled
 			var name = githubUserData.name || githubUsername;
 			var project = getProject();
 			var curDate = new Date();
@@ -320,18 +309,17 @@ ${userReason}`;
 		});
 	}
 
-	// write PRs Reviewed
 	function writeGithubPrsReviews() {
-		console.log("Starting to process PR reviews");
 		var items = githubPrsReviewData.items;
 
 		reviewedPrsArray = [];
 		githubPrsReviewDataProcessed = {};
 
 		for (var i = 0; i < items.length; i++) {
 			var item = items[i];
-			//skip if its your own pr
+			
 			if (item.user.login === githubUsername) {
+				// skips own pr from review
 				continue;
 			}
 
@@ -393,18 +381,14 @@ ${userReason}`;
 			}
 			repoLi += '</li>';
 			reviewedPrsArray.push(repoLi);
-			console.log(`Added repo ${repo} to reviewedPrsArray`);
 		}
 
 		writeScrumBody(); 
 	}
 	function writeGithubIssuesPrs() {
-		console.log("Starting to process issues/PRs");
 		var data = githubIssuesData;
-		console.log("Total items to process:", data.items.length);
 		var items = data.items;
 
-		// Reset arrays at the start
 		lastWeekArray = [];
 		nextWeekArray = [];
 
@@ -427,7 +411,6 @@ ${userReason}`;
 				if (item.state === 'open' && item.body && item.body.toUpperCase().indexOf('YES') > 0) {
 					var li2 = `<li><i>(${project})</i> - Work on Issue(#${number}) - <a href='${html_url}'>${title}</a> ${issue_opened_button}</li>`;
 					nextWeekArray.push(li2);
-					console.log("Added to nextWeekArray (contains YES)");
 				}
 				if (item.state === 'open') {
 					li = `<li><i>(${project})</i> - Opened Issue(#${number}) - <a href='${html_url}'>${title}</a> ${issue_opened_button}</li>`;
@@ -440,15 +423,8 @@ ${userReason}`;
 			} else {
 			}
 		}
-		console.log("Final arrays:", {
-			lastWeekItems: lastWeekArray.length,
-			nextWeekItems: nextWeekArray.length,
-			lastWeekContents: lastWeekArray,
-			nextWeekContents: nextWeekArray
-		});
 		writeScrumBody();
 	}
-
 	var intervalBody = setInterval(() => {
 		if (!window.emailClientAdapter) return;
 
@@ -480,13 +456,12 @@ ${userReason}`;
 		}
 	}, 500);
 }
-allIncluded('email');  // Auto-trigger on page load
+allIncluded('email'); 
 $('button>span:contains(New conversation)').parent('button').click(() => {
-    allIncluded();  // Auto-trigger on new conversation
+    allIncluded(); 
 });
 
 window.generateScrumReport = function() {
-	console.log('generateScrumReport called');
     allIncluded('popup');
 };