Deprecate remove and add permissions in ModuleAccount

[colin-axner]

addresses problem mentioned in this issue.

• Targeted PR against correct branch (see CONTRIBUTING.md)

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

• Wrote tests

• Updated relevant documentation (docs/)

• Added a relevant changelog entry: clog add [section] [-t <tag>] [-m <msg>]

• Re-reviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[codecov]

Codecov Report

Merging #4798 into master will increase coverage by 3.35%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4798      +/-   ##
==========================================
+ Coverage    50.5%   53.86%   +3.35%     
==========================================
  Files         288      270      -18     
  Lines       18516    17222    -1294     
==========================================
- Hits         9351     9276      -75     
+ Misses       8480     7262    -1218     
+ Partials      685      684       -1

[fedekunze]

@alexanderbez do we want to cherry-pick this one too ?

[alexanderbez]

ACK

[alexanderbez]

@alexanderbez do we want to cherry-pick this one too ?

Done :)