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

Roles, Users, and Permissions in MTA for the mta-documentation repo #25

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

RichardHoch
Copy link
Collaborator

@RichardHoch RichardHoch commented Jul 31, 2024

MTA 7.1.0 (mta-documentation repo)

Resolves: https://issues.redhat.com/browse/MTA-3244 for MTA 7.1 by adding a description of the three roles used in MTA: Administrator, Architect, and Migrator. Also changes discussion of persistent volume requirements, following the removal of pathfinder.

Previews:

Note: This is a copy of windup/windup-documentation#902 for the mta-documentation repo.

@RichardHoch
Copy link
Collaborator Author

@dymurray Please review this PR. I'll be asking Shveta and Ilanit to review it as well, but I cannot tag them here.

@RichardHoch RichardHoch changed the title Roles, Users, and Permissions in MTV for mta-documentation repo Roles, Users, and Permissions in MTV for the mta-documentation repo Aug 7, 2024
@RichardHoch RichardHoch changed the title Roles, Users, and Permissions in MTV for the mta-documentation repo Roles, Users, and Permissions in MTA for the mta-documentation repo Aug 7, 2024
@dymurray
Copy link
Contributor

dymurray commented Aug 7, 2024

Aside from what I found this LGTM

@RichardHoch
Copy link
Collaborator Author

@dymurray I made the changes you suggested. Please review the PR again. Thanks!

@RichardHoch
Copy link
Collaborator Author

@mguetta1 I made the change you requested. Please review this PR again.

Copy link
Collaborator

@mguetta1 mguetta1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@dymurray dymurray left a comment

Choose a reason for hiding this comment

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

LGTM

@RichardHoch RichardHoch force-pushed the 71_personas branch 2 times, most recently from a816801 to 46cfbc8 Compare August 14, 2024 13:14
@RichardHoch
Copy link
Collaborator Author

@nachandr Fixed; please review.

Copy link
Collaborator

@nachandr nachandr left a comment

Choose a reason for hiding this comment

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

LGTM

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.

5 participants