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

docs: *_invitation life cycle clarification #756

Merged
merged 6 commits into from
Jun 17, 2022

Conversation

morey-tech
Copy link
Contributor

@morey-tech morey-tech commented Jun 15, 2022

Description

Update the docs to clarify how the lifecycle of the org_invitation and project_invitation resources work. Indicating that once the invite is accepted the resource is deleted.

Link to any related issue(s):

#633 (comment)

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

Further comments

@morey-tech morey-tech requested a review from themantissa as a code owner June 15, 2022 15:09
@themantissa
Copy link
Collaborator

@morey-tech thank you for the submission. We do have a statement in these docs pages about how invitations work:

NOTE: Possible provider behavior depending on the invitee's action:

  • If the user has not yet accepted the invitation, the provider leaves the invitation as is.
  • If the user has accepted the invitation and is now an organization member, the provider will remove the invitation from the Terraform state. The invitation must then be removed from the Terraform resource configuration.
  • If the user accepts the invitation and then leaves the organization, the provider will re-add the invitation if the resource definition is not removed from the Terraform configuration.

What if we move this up and better clarify some of the points you are making this this PR do you think that will be a bit better? This obviously is an area that needs better clarification.

@morey-tech morey-tech changed the title docs: *_invite life cycle clarification docs: *_invitation life cycle clarification Jun 16, 2022
@morey-tech
Copy link
Contributor Author

@morey-tech thank you for the submission. We do have a statement in these docs pages about how invitations work:

NOTE: Possible provider behavior depending on the invitee's action:

  • If the user has not yet accepted the invitation, the provider leaves the invitation as is.
  • If the user has accepted the invitation and is now an organization member, the provider will remove the invitation from the Terraform state. The invitation must then be removed from the Terraform resource configuration.
  • If the user accepts the invitation and then leaves the organization, the provider will re-add the invitation if the resource definition is not removed from the Terraform configuration.

What if we move this up and better clarify some of the points you are making this PR do you think that will be a bit better? This obviously is an area that needs better clarification.

Thanks for pointing that out! I agree that moving it up is important to bring it to the user's attention earlier. This is also typical of resources that may behave differently than a user might assume (i.e., being removed from the state outside without the resource needing to be destroyed). I also think including these notes in the project_invite resource docs and not only the org_invite would be helpful.

I have updated the PR to integrate the existing note in the org_invitiation docs into the changes I am proposing. Please let me know what you think.

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.

Just a small adjustment to make this a bit more clear.

website/docs/r/project_invitation.html.markdown Outdated Show resolved Hide resolved
website/docs/r/org_invitation.html.markdown Outdated Show resolved Hide resolved
Thanks for the suggestions!

Co-authored-by: Melissa Plunkett <[email protected]>
@morey-tech morey-tech requested a review from themantissa June 17, 2022 11:44
@themantissa
Copy link
Collaborator

Thanks again @morey-tech ! These changes will go out with our next release.

@themantissa themantissa merged commit 80d231a into mongodb:master Jun 17, 2022
@morey-tech morey-tech deleted the invite-docs-clarification branch June 17, 2022 20:56
guineveresaenger pushed a commit to pulumi/terraform-provider-mongodbatlas that referenced this pull request Sep 13, 2022
* docs(project_invitation): add notes on invite life cycle

* docs(org_invitation): add notes on invite life cycle

* docs(org_invitation): integrate existing note

* docs(project_invitation): match org invitation note

* docs(org_invitation): fix reference to project

* doc: clarify clarification for more clarity

Thanks for the suggestions!

Co-authored-by: Melissa Plunkett <[email protected]>

Co-authored-by: Melissa Plunkett <[email protected]>
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