-
Notifications
You must be signed in to change notification settings - Fork 19
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
Bug : Standardize Promise Handling #15
Comments
I am excited to work on this @Aryainguz Can you assign me this for SWOC24? |
Great observation and thought @mathanraj0601 go ahead and feel free to reach out for any help! |
That's great @Aryainguz |
Hey @Aryainguz 🌟 I've submitted a PR for the task mentioned . While diving into the issue, I spotted some opportunities to enhance the code for better readability and error handling. So, I've create another issue or those improvements. Could you kindly assign me to the linked issue (#29 ) ? I'm eager to kickstart the work and contribute to the project! |
Solve fixes : #15 : Fixed async operation
standardize Promise Handling
In our codebase, there is a mix of using
await
andthen/catch
for handling promises. We need to standardize the approach to improve code consistency and maintainability.Plan:
The text was updated successfully, but these errors were encountered: