We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
When you are reading this code can you quickly tell what false and true as argument stand for?
false
true
Telegram-bot-schedule/src/buttons/buttonsHandler.js
Lines 19 to 21 in 8b7a1c4
Basically, booleans in the function params should be avoided and replaced with options object, i.e. check the difference between calls
options
makeRequest('/api/data' , true)
makeRequest('/api/data' , { raw: true }
The second call looks much more readable.
I suggest: Find all boolean params in code and replace them with options object
The text was updated successfully, but these errors were encountered:
Btw, in some low level or performance dependent api's it is normal to sacrifice some readability. Check this comment in the vue core repo for example
Sorry, something went wrong.
fbbf6b5
No branches or pull requests
When you are reading this code can you quickly tell what
false
andtrue
as argument stand for?Telegram-bot-schedule/src/buttons/buttonsHandler.js
Lines 19 to 21 in 8b7a1c4
Basically, booleans in the function params should be avoided and replaced with
options
object, i.e. check the difference between callsmakeRequest('/api/data' , true)
makeRequest('/api/data' , { raw: true }
The second call looks much more readable.
I suggest:
Find all boolean params in code and replace them with
options
objectThe text was updated successfully, but these errors were encountered: