Skip to content

Add support for note field in vercel_firewall_bypass#406

Merged
dglsparsons merged 2 commits intovercel:mainfrom
SIGTERM-015:main
Oct 13, 2025
Merged

Add support for note field in vercel_firewall_bypass#406
dglsparsons merged 2 commits intovercel:mainfrom
SIGTERM-015:main

Conversation

@SIGTERM-015
Copy link
Copy Markdown
Contributor

@SIGTERM-015 SIGTERM-015 commented Oct 13, 2025

In the Vercel API the field note is supported but not in the resource of the provider.

@dglsparsons dglsparsons merged commit 24cce00 into vercel:main Oct 13, 2025
1 check passed
@SIGTERM-015
Copy link
Copy Markdown
Contributor Author

Hello @dglsparsons the terrafor language server is detecting that the note field should not be present. Did i miss something in my pr?
image

@SIGTERM-015
Copy link
Copy Markdown
Contributor Author

just to clarify I updated the provider and terraform validate command sais the config is valid. it is just the language server

@dglsparsons
Copy link
Copy Markdown
Collaborator

I wonder if the LSP is just using a cached, and old, version of the vercel/vercel provider, or pulling directly from the docs.

restarting LSP may help, or making sure you've run terraform init -upgrade. Otherwise, i guess it'll fix itself in time. Your PR was totally correct.

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