react-notion-x icon indicating copy to clipboard operation
react-notion-x copied to clipboard

notion-client search api broken

Open seonglae opened this issue 2 years ago • 14 comments
trafficstars

Description

notion-client search api stopped workine with something like this error

{
  errorId: 'a35d346e-2540-4178-900d-3e1267ec727d',
  name: 'ValidationError',
  message: 'Invalid input.'
}

Notion Test Page ID

here doesn't work image

seonglae avatar Jun 12 '23 17:06 seonglae

Hi @seonglae , I have resolved this issue, and will create a pull request later. @transitive-bullshit FYI

dongtong avatar Jun 24 '23 18:06 dongtong

@dongtong Can you explain what was the problem?

seonglae avatar Jun 25 '23 03:06 seonglae

I tried some fixes but couldn't resolved it. So I just thought that Notion blocked request except official Notion API

seonglae avatar Jun 25 '23 04:06 seonglae

@dongtong any updates about the fix?

chervonnyy avatar Jun 26 '23 08:06 chervonnyy

@chervonnyy @seonglae Sorry, I was busy recently. @transitive-bullshit Do we have pr template in this project?

dongtong avatar Jul 04 '23 01:07 dongtong

@dongtong https://github.com/NotionX/react-notion-x/blob/master/.github/pull_request_template.md

seonglae avatar Jul 04 '23 04:07 seonglae

@dongtong Any updates?

kaichii avatar Jul 04 '23 11:07 kaichii

Created a pull request to fix this 👉 https://github.com/NotionX/react-notion-x/pull/505

Armster15 avatar Jul 07 '23 20:07 Armster15

In the meantime while we wait for the pull request to be (hopefully) merged, you can use patch-package and the following patch file to fix the issue:

notion-client+6.15.6.patch

Armster15 avatar Jul 07 '23 20:07 Armster15

I forked this repository for my project Texonom and applied @Armster15 's solution to @texonom/nclient If someone doesn't want to apply patch-package, you can use it too

seonglae avatar Jul 08 '23 07:07 seonglae

In the meantime while we wait for the pull request to be (hopefully) merged, you can use patch-package and the following patch file to fix the issue:

notion-client+6.15.6.patch

I tried like this and it works now, thank you!! And I hope this PR(#505 ) will be merged soon. 🙏 @transitive-bullshit

stefan-ysh avatar Aug 08 '23 04:08 stefan-ysh

I think we just need a tag and release @transitive-bullshit

jacksongoode avatar Aug 21 '23 19:08 jacksongoode

any updates here? think we just need a release

lisposter avatar Nov 19 '23 03:11 lisposter

@lisposter @jacksongoode If you want to fix rn, you can use https://www.npmjs.com/package/@texonom/nclient for compatible package This is live example that search api is fixed

seonglae avatar Nov 19 '23 04:11 seonglae