Skip to content
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

Remove node-fetch from Client.ts #460

Closed
wants to merge 291 commits into from
Closed

Conversation

lcswillems
Copy link

@lcswillems lcswillems commented Sep 30, 2023

Fixes #459

aoberoi and others added 30 commits May 13, 2021 03:03
this emits a warning from ava@typescipt which can be safely ignored
This adds a warning in npm install for ava@typescript that can be safely ignored
rhart92 and others added 26 commits April 12, 2023 13:36
* add block text example

* clean up comments

* add embed option

* update comment:

* pr feeedback: name change, updated switch case, join richtext array without comma

* use iteratePaginatedAPI helper to capture all blocks on page
* Wiki related changes

* Remove oauth related endpoints
* add fullstack webform example app

* update package.json detail

* make linter happy

* delete old folder (name change

* Update examples/web-form-with-express/public/client.js

Co-authored-by: Ron Yehoshua <ron@makenotion.com>

* update var naming for improved readbility

* add console back to match appendApiResponse()

---------

Co-authored-by: Ron Yehoshua <ron@makenotion.com>
* update readme with isfullpageordatabase

* fix spacing

* Fix formatting.

---------

Co-authored-by: Ryan Hart <ryan@makenotion.com>
* add new example (wip)

* update readme

* add intermediate example

* add more comments and resources to each example

* add additional resources section

* rename some files for consistency, clean up readme, more console logs

* make linter happy

* readme updates

* clean up links

* add console for querying
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@rhart92
Copy link
Contributor

rhart92 commented Oct 3, 2023

Unfortunately we had to remove an old .env file from the repo which required completely re-writing the commit history causing this PR's commit history to be out of date. Please re-submit this PR after pulling a fresh copy of this repo 🙏 Apologies for any inconvenience.

@rhart92 rhart92 closed this Oct 3, 2023
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.

Remove dependency on node-fetch