Skip to content
This repository has been archived by the owner on Jun 28, 2024. It is now read-only.

[BUG] Required Notion-Version header is missing in maps #337

Open
zdne opened this issue Jul 19, 2023 · 0 comments
Open

[BUG] Required Notion-Version header is missing in maps #337

zdne opened this issue Jul 19, 2023 · 0 comments
Labels
AI-related Issue is related to use of AI in our BE

Comments

@zdne
Copy link
Member

zdne commented Jul 19, 2023

Following the notion tutorial everything works fine. But when I try some other usecases like:

superface new notion "create page"

or

superface new notion "search for a page"

I get a correct map that is missing the mandatory Notion-Version header. Note, with "list all users" the header is set correctly.

Expected Behavior

When I create unsafe usecase for notion the Notion-Version header is set in Comlink maps:

function SearchForPage({input, parameters, services}){
  const url = `${services.default}/v1/search`;
const options = {
  method: 'POST',
  body: {
    query: input.query,
    filter: input.filter,
    sort: input.sort,
    start_cursor: input.start_cursor,
    page_size: input.page_size,
  },
  headers: {
    'Content-Type': 'application/json',
    'Notion-Version': '2022-06-28',
  },

Current Behavior

The header 'Notion-Version': '2022-06-28' is not present in maps for unsafe notion usecases

Steps to Reproduce

superface new notion "search for a page"

Your Environment

  • macOS
  • Safari
  • Node v18.13.0
@Jakub-Vacek Jakub-Vacek added the AI-related Issue is related to use of AI in our BE label Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AI-related Issue is related to use of AI in our BE
Projects
None yet
Development

No branches or pull requests

2 participants