Skip to content

Update contend.md #1241

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
merged 2 commits into from
Aug 3, 2023
Merged

Update contend.md #1241

merged 2 commits into from
Aug 3, 2023

Conversation

Fr4ng
Copy link
Collaborator

@Fr4ng Fr4ng commented Aug 3, 2023

Added a sentence in Member Types to specify muliple admins allowed

What This PR Changes

  • (Please explain here why you created the pull request and specify what it changes)

Contribution Guidelines

Added a sentence in Member Types to specify muliple admins allowed
Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

Changes requested:

  • Contemplate the possibility of removing the "thus" phrase

@gatsby-cloud
Copy link

gatsby-cloud bot commented Aug 3, 2023

✅ docs-content deploy preview ready

Updated according to reviewer's feedback: sentence removed
Copy link
Contributor

@martab1994 martab1994 left a comment

Choose a reason for hiding this comment

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

Perfect, we can merge thanks!

Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

Approved

waiting for linter checks and gatsnt preview

I will merge

@jcarolinares
Copy link
Contributor

image
Looking good

@jcarolinares jcarolinares merged commit 18d70ad into main Aug 3, 2023
@jcarolinares jcarolinares deleted the Fr4ng-admin-review branch August 3, 2023 09:31
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.

3 participants