-
Notifications
You must be signed in to change notification settings - Fork 2
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
inject request method #173
Conversation
@@ -57,6 +57,7 @@ class PraxInjection { | |||
private stateEvents = new EventTarget(); | |||
|
|||
private injection: Readonly<PenumbraProvider> = Object.freeze({ | |||
request: () => this.postConnectRequest().then(() => Promise.resolve()), |
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.
issue (blocking): can you add code documentation on the fact that this is temporary and should be removed after a pd point release?
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.
commented
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.
this was approved by another review, but you've left a blocking comment, so i will avoid merging this unless you re-review.
go ahead and merge this if you intend to include it in a release.
i think this should be part of a fix release, rather than waiting on another solution. |
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.
Merging to include in next release
temporary fix to mitigate missing request method