You’ve signed the CLA, OsamaSayegh. Thank you! This pull request is ready for review.
Trivial but this could be one line if you like
raise Discourse::InvalidAccess if BackupRestore.backups_disallowed?
Another trivial change but redis has an
exists method which is slightly simpler.
Is this site setting the only way to trigger this? I would prefer if it could be passed as an option rather than setting a global setting like this.
I think this PR can be closed because Neil fixed the underlying problem this PR was meant to fix?