Skip to content
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

Whatsapp imported file error fixed #11

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Prashun123
Copy link

@Prashun123 Prashun123 commented Dec 7, 2021

Closes #8
The error we received while using the imported file of Whatsapp is fixed, it was giving some error due to some regex expressions, but they are fixed now.

@JRavi2
Copy link
Owner

JRavi2 commented Dec 7, 2021

@Prashun123 Thanks for the PR!

This does fix the bug, but, will also cause the program to not recognize the previous formats (try running the tests).
A simple fix for this would be to make ampm optional in the Regex.

Also, while creating the dictionary maintain a separate variable for time format, and set this variable to the appropriate value depending on whether there is an ampm in match.

Finally, you can check whether the fix worked or not by running the tests :)

@JRavi2
Copy link
Owner

JRavi2 commented Dec 15, 2021

@Prashun123 Any updates on this?

@Prashun123
Copy link
Author

Sir, there were some submissions which incorporated the date and time issue more correctly, so I thought to start work on other issue's.

@JRavi2
Copy link
Owner

JRavi2 commented Dec 17, 2021

#10 Is currently is focused on fixing the dates, you can continue working on this if you want :)

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.

Bug generated while importing whatsapp chat
2 participants