add information about new errors for revoke commands in the additions/removals section #1170
Conversation
…/removals section
|
This PR includes documentation updates Updated pages: |
| REVOKE ROLE regularUser FROM bob | ||
| ---- | ||
| a| | ||
| Notifications for impossible `REVOKE` commands, where a user, a role, or a database does not exist, have been replaced with errors. |
There was a problem hiding this comment.
I feel like maybe it should be under removed features instead of updated features, we did remove the ability to not get an error...
There was a problem hiding this comment.
hmm, maybe. Feels more like a change than an removal (since we are changing the notification into an error). But I don't have a strong opinion 🤷
There was a problem hiding this comment.
Had we not had a notification before this would absolutely be a case of removed capabilities -> introducing an error, so I think I'd go with that here as well 🤔 errors are not really part of the features capabilities but rather things it can't do (and the updates section is usually for additional capabilities, not restricting it I think)
But we could see if the docs team has any other view on things
Operations manual PR