search

ahmedkhan1 / email-client-app

This project is designed to sync emails from various providers (currently Outlook) to an Elasticsearch database, providing real-time updates.
0 stars 0 forks source link

Not Valid solution #2

Open bokrma opened 3 months ago

bokrma commented 3 months ago

Hello! as i can see from the first view, you fetched the emails and kept the synced data using corn jobs, which is not a valid solution for a real-life email client application please find a better way (and test it), to ensure the data is always synced use case you could have 10000 email accounts that are syncing the data (near realtime/ realtime is the most important part of the sync process)

ahmedkhan1 commented 3 months ago

Do you want me to update this solution for you to recheck or is the submission closed now?

bokrma commented 3 months ago

you can still update the solution, the submission not closed but please be sure its a real-life valid solution when its gonna apply on real app

ahmedkhan1 commented 3 months ago

Alright let me update the code.

ahmedkhan1 commented 3 months ago

@bokrma Hi I have created a pull request for the changes. Please check and let me know.