-
Notifications
You must be signed in to change notification settings - Fork 599
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
Support oauth token endpoint, add wiki verification support, fix issue #420 #438
Support oauth token endpoint, add wiki verification support, fix issue #420 #438
Conversation
…with toggleable heading type
verified_by: null | ||
} | ||
|
||
type VerificationPropertyResponse = { |
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.
is it auto-generated?
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.
Yes
@@ -525,6 +553,29 @@ export default class Client { | |||
}) | |||
} | |||
|
|||
public readonly oauth = { |
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.
what does this do?
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.
Allows you to call the token
endpoint.
const { Client } = require("@notionhq/client")
// Initializing a client
const notion = new Client({ ... })
const tokenResponse = await notion.oauth.token({ ... })
This is similar to how we nest all the block
methods under a blocks
object.
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.
hmm... we were not allowed to call the token
endpoint before your change?
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.
Not through the SDK 😓 See #73
…with toggleable heading type (#438)
Fixes #420 and #73.