Skip to content

Conversation

Fr4ng
Copy link
Contributor

@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

3 participants