-
Notifications
You must be signed in to change notification settings - Fork 7.6k
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
Valid Notion page IDs are rejected #10831
Comments
Hey @nihaals, We have created an internal ticket to look into this which we will be tracking as "GHC-235" |
Hey @nihaals, Can you share some URLs that show the errors? Notion page urls should be UUID 4 looking at their documentation. |
I think this might have changed within the past couple of weeks as the first page I just made wasn't a UUIDv4: https://www.notion.so/10336a9a827580b895e3f7ff1d3ea585. |
Yes, we are having the exact same issue here |
@jshay21 @helbertparanhos can you both upgrade to the newest release and see if you have the same issue, @helbertparanhos your issue looks to be different as Notion is returning a different error so it is likely that you have an issue in your workflow. |
Yes, the update fixed it. Thank you
…On Thu Sep 26, 2024, 05:12 PM GMT, Jon ***@***.***> wrote:
@jshay21 <https://github.com/jshay21> @helbertparanhos <https://github.com/helbertparanhos> can you both upgrade to the newest release and see if you have the same issue, @helbertparanhos <https://github.com/helbertparanhos> your issue looks to be different as Notion is returning a different error so it is likely that you have an issue in your workflow.
—
Reply to this email directly, view it on GitHub <#10831 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABI2GNMTXYCGXSSLISTMHTDZYQ56LAVCNFSM6AAAAABOJCYICKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZXGUYDGOJTGY>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@Joffcom #10860 didn't fix this issue (see #10860 (comment)). Here's a simple workflow that reproduces the issue ("ERROR: Page parameter's value is invalid. This is likely because the URL entered is incorrect") on v1.60.1 using the ID from #10831 (comment): Workflow JSON{
"nodes": [
{
"parameters": {},
"id": "682aca11-cd31-4e04-a1a0-e82df830b2e2",
"name": "Trigger",
"type": "n8n-nodes-base.manualTrigger",
"typeVersion": 1,
"position": [
580,
280
]
},
{
"parameters": {
"assignments": {
"assignments": [
{
"id": "4f6cbbc6-562b-48a1-8d09-b0814671b5f6",
"name": "id",
"value": "10336a9a827580b895e3f7ff1d3ea585",
"type": "string"
}
]
},
"options": {}
},
"id": "785effe4-b5fb-40ca-af04-6b662b5c846c",
"name": "Set ID",
"type": "n8n-nodes-base.set",
"typeVersion": 3.4,
"position": [
760,
280
]
},
{
"parameters": {
"operation": "archive",
"pageId": {
"__rl": true,
"value": "={{ $json.id }}",
"mode": "id"
}
},
"id": "baf6f8e3-69e2-4e62-a8e1-ffc7ac788319",
"name": "Delete Notion page",
"type": "n8n-nodes-base.notion",
"typeVersion": 2.2,
"position": [
940,
280
]
}
],
"connections": {
"Trigger": {
"main": [
[
{
"node": "Set ID",
"type": "main",
"index": 0
}
]
]
},
"Set ID": {
"main": [
[
{
"node": "Delete Notion page",
"type": "main",
"index": 0
}
]
]
}
},
"pinData": {}
} |
Ah, I thought you meant latest stable and didn't see that PR linking to this issue. |
Bug Description
Notion documents page IDs as being UUIDv4, however I have some page IDs that don’t have a 4 in the correct position. This means some Notion nodes reject the input even though the ID came directly from a Notion API response.
n8n’s regexes that neither the ID or the URL from the API response match:
n8n/packages/nodes-base/nodes/Notion/shared/descriptions/PageDescription.ts
Lines 87 to 128 in cef6432
To Reproduce
Expected behavior
The page is archived.
Operating System
Docker n8n:1.56.2
n8n Version
1.56.2
Node.js Version
Docker
Database
SQLite (default)
Execution mode
main (default)
The text was updated successfully, but these errors were encountered: