You are viewing a single comment's thread from:
RE: Keychainify on direct access to a SteemConnect URL
Thank you for your contribution!.
- L74 which could be refactored using Array.includes
- We probably can rewrite Promise as async/await see here
- I prefer
string.includesoverstring.indexOf()===-1
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
Thanks @justyy
I prefer using indexOf() in cases like here for more compatibility. As for async/await, my function calls another one that uses callbacks so using Promises makes it easier for calling resolve/reject
Thank you for your review, @justyy! Keep up the good work!