Skip to content
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

Notes are dropped from nested attributes in docs #530

Closed
Tracked by #1794
jkodroff opened this issue May 26, 2022 · 4 comments
Closed
Tracked by #1794

Notes are dropped from nested attributes in docs #530

jkodroff opened this issue May 26, 2022 · 4 comments
Assignees
Labels
area/docsgen Issues with docs capture or example rendering, historically part of pkg/tfgen kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@jkodroff
Copy link
Member

In pulumi/pulumi-aws#1969, the notes toward the bottom of https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/autoscaling_group#instance_refresh ("A refresh is started any of the following...") are dropped which is causing confusion for our users.

(We still need to find the root cause and determine the effort to fix.)

@jkodroff jkodroff added kind/bug Some behavior is incorrect or out of spec area/docs Improvements or additions to docs for this repo labels May 26, 2022
@jkodroff jkodroff self-assigned this May 31, 2022
@jkodroff jkodroff removed their assignment Aug 15, 2022
@t0yv0 t0yv0 added area/docsgen Issues with docs capture or example rendering, historically part of pkg/tfgen and removed area/docs Improvements or additions to docs for this repo labels Apr 5, 2023
@jkodroff
Copy link
Member Author

jkodroff commented Jan 9, 2024

This appears to be at least partially fixed:

image

I'm not sure if the content is supposed to be in the triggers field as the upstream docs seem to list the notes as part of the supporting type itself, rather than a part of a specific field in that type, but it's on the page now, and that's much better!

image

See: https://www.pulumi.com/registry/packages/aws/api-docs/autoscaling/group/#triggers_nodejs

@guineveresaenger
Copy link
Contributor

Hi @jkodroff - have you seen any recent instances of notes missing entirely?

You're right that ideally, these notes should be associated with the top-level GroupInstanceRefresh. That is a slightly different bug unfortunately, one that is related to the brittle way we do docs discovery. I'll file a separate issue for this, but my feeling is that that would best be addressed by rethinking how we transform our docs in the first place.

@jkodroff
Copy link
Member Author

They're definitely not dropped, so I think that it's reasonable to close this issue.

FWIW, IME the notes displaying under triggers instead of afterwards is a minor usability issue.

I can't close b/c I don't have perms to add a resolution tag, but ok to close.

@guineveresaenger
Copy link
Contributor

I'm closing this issue as fixed. Notes are no longer dropped, but they are misaligned.

I filed #2086 to track the misalignment of Note sections.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docsgen Issues with docs capture or example rendering, historically part of pkg/tfgen kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

No branches or pull requests

3 participants