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
feature/sc-121448/configure-typescript-with-just-checkjs #170
feature/sc-121448/configure-typescript-with-just-checkjs #170
Conversation
1937c94
to
cf2b7bc
Compare
d41e0ec
to
a29b473
Compare
32c7861
to
1b939e4
Compare
@@ -75,7 +83,9 @@ class Particle { | |||
username, | |||
password, | |||
grant_type: 'password', | |||
// @ts-ignore |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm curious why these ts-ignore are necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The ones related to this
are because they are set by doing Object.assign(this, Defaults, options);
but they have no explicit this.clientId
initialization so it was complaining of this
not having said properties.
I thought of adding default values before the object.assign but wasn't sure which values to give it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Adds typescript configurations to run
checkJS
and fixes docs errors