-
Notifications
You must be signed in to change notification settings - Fork 49
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
Refactor duplicate code in src/messaging/data.js #92
base: f24
Are you sure you want to change the base?
Refactor duplicate code in src/messaging/data.js #92
Commits on Aug 29, 2024
-
Refactor: Remove duplicated code blocks in messaging logic
Consolidated identical code blocks in the conditional structure of src/messaging/data.js to improve maintainability. The repeated assignment of message.newSet = true was refactored into a single condition to reduce redundancy and enhance code clarity.
Configuration menu - View commit details
-
Copy full SHA for 4e010ad - Browse repository at this point
Copy the full SHA 4e010adView commit details
Commits on Sep 3, 2024
-
Configuration menu - View commit details
-
Copy full SHA for b037b0d - Browse repository at this point
Copy the full SHA b037b0dView commit details
Commits on Sep 5, 2024
-
i refactored the code in src/messaging/data.js to remove the duplicat…
…e. I also added the original contents of the file widgets.js
Configuration menu - View commit details
-
Copy full SHA for 41cda07 - Browse repository at this point
Copy the full SHA 41cda07View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5bab71c - Browse repository at this point
Copy the full SHA 5bab71cView commit details -
i enhanced the code again by removing duplicates and also added new c…
…omments from line 81 to 90
Configuration menu - View commit details
-
Copy full SHA for 2d4cf1e - Browse repository at this point
Copy the full SHA 2d4cf1eView commit details -
i refactored the code by removing the duplicate code block from line …
…84 to line 91 of the file data.js in the src/messaging
Configuration menu - View commit details
-
Copy full SHA for 09324ab - Browse repository at this point
Copy the full SHA 09324abView commit details -
i changed the syntax code and i refactored the code by removing the …
…duplicate code block from line 84 to line 91 of the file data.js in the src/messaging
Configuration menu - View commit details
-
Copy full SHA for e2d7852 - Browse repository at this point
Copy the full SHA e2d7852View commit details -
Configuration menu - View commit details
-
Copy full SHA for 422ec04 - Browse repository at this point
Copy the full SHA 422ec04View commit details -
I removed the duplicate code from line 83 to 89 of the file data.js. …
…added comments and tested the new code chunck
Configuration menu - View commit details
-
Copy full SHA for f6c8d81 - Browse repository at this point
Copy the full SHA f6c8d81View commit details -
fix: correct indentation in data.js
- Adjusted indentation levels to match the expected number of tabs. - Ensured consistent formatting for better readability and maintainability. Affected lines: - Line 84: Increased indentation from 3 tabs to 4 tabs. - Line 89: Increased indentation from 4 tabs to 5 tabs. - Line 92: Increased indentation from 5 tabs to 6 tabs. - Line 93: Increased indentation from 4 tabs to 5 tabs. - Line 94: Increased indentation from 3 tabs to 4 tabs. - Line 95: Increased indentation from 3 tabs to 4 tabs.
Configuration menu - View commit details
-
Copy full SHA for 2f727cb - Browse repository at this point
Copy the full SHA 2f727cbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7e5d83d - Browse repository at this point
Copy the full SHA 7e5d83dView commit details