PG-1523 Rework uninstallation documentation cleanup #490
Conversation
This commit rearranges the uninstallation documentation and adds instructions for disabling WAL encryption.
| To proceed, you must have one of the following privileges: | ||
|
|
||
| - Superuser privileges (to remove the extension globally), or | ||
| - Database owner privileges (to remove it from a specific database only) |
There was a problem hiding this comment.
Not sure what this adds. Under most circumstances you need to be superuser and to me it is so extremely obvious.
There was a problem hiding this comment.
This is mostly to ensure that new users are aware of the necessary admin privileges, I am trying to include, going forward, as much information as possible for all types of users. Sometimes, some information that seems repeating or redundant would still be appreciated for some users.
Let me know your thoughts. :)
There was a problem hiding this comment.
Why do you have ()? Without them, it reads exactly the same
There was a problem hiding this comment.
I used the () to explain a bit why privileges are required, I could also use commas but I feel these pop out better when reading them
There was a problem hiding this comment.
IMHO, the text without brackets explains the same information as well as with them:) But () make it heavier. Up to you, though
| To proceed, you must have one of the following privileges: | ||
|
|
||
| - Superuser privileges (to remove the extension globally), or | ||
| - Database owner privileges (to remove it from a specific database only) |
There was a problem hiding this comment.
Why do you have ()? Without them, it reads exactly the same
This commit cleans up the uninstallation documentation.