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

Fix Dynamic Resource Providers Docs #3515

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

meagancojocar
Copy link
Contributor

Description

I found it really deceptive that these docs say its supported in all languages and show code examples in them all but then the chooser is blank and we dont actually support it in all languages.

CleanShot 2023-10-18 at 17 11 20

Checklist:

  • I have reviewed the style guide.
  • If blogging, I have reviewed the blogging guide.
  • I have manually confirmed that all new links work.
  • I added aliases (i.e., redirects) for all filename changes.
  • If making css changes, I rebuilt the bundle.

@interurban interurban added the area/docs Improvements or additions to documentation label Oct 24, 2023
@interurban
Copy link
Collaborator

Hey @meagancojocar sorry for being behind; I'm looking into where our PR review links went for these and will approve shortly

@@ -50,7 +50,6 @@ const myProvider: pulumi.dynamic.ResourceProvider = {
}
```

{{% /choosable %}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be put back in I think. (Should fix your build error.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants