Skip to content

docs(website): fix federated_settings_org_config resource name by removing the misleading trailing s #908

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

Merged

Conversation

Horgix
Copy link
Contributor

@Horgix Horgix commented Nov 9, 2022

Description

Due to the file name, the resource federated_settings_org_config is actually caled federated_settings_org_configs in the Terraform Registry documentation (notice the superfluous trailing s).

2022-11-09-135813_1052x512_scrot

This simply renames the file to fix this.

Type of change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Documentation fix/enhancement

Required Checklist:

  • I have signed the MongoDB CLA
  • I have read the Terraform contribution guidelines
  • I have added tests that prove my fix is effective or that my feature works per HashiCorp requirements
  • I have added any necessary documentation (if appropriate)
  • I have run make fmt and formatted my code

I didn't find any clean way to "test" the website part so I left that unchecked. I'd be happy to read any instructions on how to validate such a documentation change!

Further comments

None.

…emoving the misleading trailing `s`

Due to the file name, the resource `federated_settings_org_config` is
actually caled `federated_settings_org_configs` in the Terraform
Registry documentation (notice the superfluous trailing `s`).

This simply renames the file to fix this.
@Horgix Horgix requested a review from a team as a code owner November 9, 2022 13:00
@Horgix
Copy link
Contributor Author

Horgix commented Nov 9, 2022

CLA signed ✔️ 🙂 Approved on my company's side by @francoisfanuel

@Zuhairahmed Zuhairahmed self-requested a review November 9, 2022 20:55
Copy link
Contributor

@Zuhairahmed Zuhairahmed left a comment

Choose a reason for hiding this comment

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

Thanks @Horgix! Also confirming that you signed our CLA, this is a good catch. LGTM

Copy link
Collaborator

@andreaangiolillo andreaangiolillo left a comment

Choose a reason for hiding this comment

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

LGTM

@Horgix
Copy link
Contributor Author

Horgix commented Nov 10, 2022

Thanks for the reviews! ☺️

What's the next step to get this merged? Do I need to do anything else on my end? 🙂

Copy link
Collaborator

@themantissa themantissa left a comment

Choose a reason for hiding this comment

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

LGTM!

@Zuhairahmed Zuhairahmed merged commit 0c0bc96 into mongodb:master Nov 10, 2022
@Zuhairahmed
Copy link
Contributor

Just merged @Horgix , thanks again for your contribution!

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.

4 participants