Open
Conversation
- Created GetRandomWelcomeMessage function in greeting.py with 10 different greeting variations - Updated auth.py to use random greetings instead of static message - Includes original greeting message plus 9 new variations - All changes pass ruff linting checks Closes Code-A2Z#327
- Added detailed project structure section with visual directory tree - Documented all major directories and their purposes - Included explanations for the modular organization - Follows the suggested format from issue Code-A2Z#346 - Places the section after Features and before Contributing guidelines Closes Code-A2Z#346
… handling - Removed unreliable browser automation using pyautogui - Added comprehensive phone number validation function - Implemented proper error handling for network issues and invalid inputs - Added progress tracking and detailed user feedback - Improved CSV validation and error messages - Added configurable wait time for message sending - Implemented success/failure summary with detailed reporting - Removed TODO comment as issue is now resolved - All changes pass ruff linting checks Closes Code-A2Z#349
|
Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE. |
Avdhesh-Varshney
requested changes
Oct 29, 2025
Collaborator
Avdhesh-Varshney
left a comment
Avdhesh-Varshney
left a comment
There was a problem hiding this comment.
Attach a working video of this feature enhancement.
Take reference of video from this PR Link
Collaborator
There was a problem hiding this comment.
Revert this file changes.
Collaborator
There was a problem hiding this comment.
Revert these changes.
Collaborator
|
And Try to learn from this project without using AI. |
Collaborator
|
@Om7035 What's the update? Are you working on the fix? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes: #349
Describe the add-ons or changes you've made
Fixed the WhatsApp automation sendMsg function by removing unreliable browser automation and implementing a robust solution with comprehensive error handling and user feedback.
Key Changes:
pyautoguikeyboard simulationTechnical Improvements:
sendwhatmsg_instantly()withsendwhatmsg()for better reliabilityWorking Video 📷
[You'll need to record a video showing the improved WhatsApp automation with CSV upload, progress tracking, and success/failure reporting]
Type of change ☑️
Note to reviewers 📄
The implementation now provides:
The solution addresses all acceptance criteria from issue #349: