Skip to content

Whatsapp imported file error fixed#11

Open
Prashun123 wants to merge 2 commits intoJRavi2:mainfrom
Prashun123:main
Open

Whatsapp imported file error fixed#11
Prashun123 wants to merge 2 commits intoJRavi2:mainfrom
Prashun123:main

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