From 91bfc33312fb8024f7f41a5522b85ff7862ca028 Mon Sep 17 00:00:00 2001 From: Paine Leffler Date: Tue, 10 May 2022 00:18:39 -0500 Subject: [PATCH] fixed typo of roll to role (#5484) Co-authored-by: Paine Leffler Co-authored-by: Dominic Saadi <32992335+jtoar@users.noreply.github.com> --- docs/docs/tutorial/chapter7/rbac.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docs/docs/tutorial/chapter7/rbac.md b/docs/docs/tutorial/chapter7/rbac.md index 5b7f7dab26d7..70a51c20459e 100644 --- a/docs/docs/tutorial/chapter7/rbac.md +++ b/docs/docs/tutorial/chapter7/rbac.md @@ -1244,7 +1244,7 @@ Having a role like "admin" implies that they can do everything...shouldn't they 1. Add "admin" to the list of roles in the `hasRole()` checks in components, `@requireAuth` directive, and `requireAuth()` check in services 2. Don't make any changes in the code, just give the user in the database additional roles—so admins will also have the "moderator" role in addition to "admin" -By virtue of the name "admin" it really feels like someone should only have that one single roll and be able to do everything. So in this case it might feel better to add "admin" to `hasRole()` and `requireAuth()`. +By virtue of the name "admin" it really feels like someone should only have that one single role and be able to do everything. So in this case it might feel better to add "admin" to `hasRole()` and `requireAuth()`. But, if you wanted to be more fine-grained with your roles then maybe the "admin" role should really be called "author". That way it makes it clear they only author posts, and if you want someone to be able to do both actions you can explicitly give them the "moderator" role in addition to "author."