Skip to content

change when you initialize the requestwrapper to pass along the restI… - #80

Merged
tiagosiebler merged 2 commits into
sieblyio:masterfrom
blake41:master
Feb 18, 2021
Merged

change when you initialize the requestwrapper to pass along the restI…#80
tiagosiebler merged 2 commits into
sieblyio:masterfrom
blake41:master

Conversation

@blake41

@blake41 blake41 commented Feb 18, 2021

Copy link
Copy Markdown
Contributor

change when you initialize the requestwrapper to pass along the restInverseOptions to getBaseUrl currently the ability to pass your own baseUrl is not working.

I don't know Typescript but I'm assuming this is the correct fix

…nverseOptions to getBaseUrl currently the ability to pass your own baseUrl is not working

@tiagosiebler tiagosiebler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch! I missed that in my last refactor. Can you also do this in the linear client? The param name is a bit misleading there but it works the same way.

Last but not least, please also bump the version in the package json to 2.0.1. That'll push to npm once this gets merged. Thanks!

@blake41

blake41 commented Feb 18, 2021

Copy link
Copy Markdown
Contributor Author

done!

@tiagosiebler tiagosiebler left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this!

@tiagosiebler
tiagosiebler merged commit ca05ca0 into sieblyio:master Feb 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants