fix: remove ourselves from members when leaving community

[osmaczko]

Otherwise clients see invalid members count in invitation bubbles.

[status-github-bot]

Pull Request Checklist

• Have you updated the documentation, if impacted (e.g. docs.status.im)?
• Have you tested changes with mobile?
• Have you tested changes with desktop?

[status-im-auto]

Jenkins Builds

❔	Commit	#️⃣	Finished (UTC)	Duration	Platform	Result
✔️	e3c0dea	#1	2022-09-15 12:43:25	~2 min	linux	📦zip
✔️	e3c0dea	#1	2022-09-15 12:44:58	~4 min	ios	📦zip
✔️	e3c0dea	#1	2022-09-15 12:46:53	~5 min	android	📦aar

[0x-r4bbit]

Why exactly is this called RemoveOurselvesFromOrg ?
This seems a pretty generic function. In fact it seems to be just an exposed API for removeMemberFromOrg.

[osmaczko]

If you compare RemoveOurselvesFromOrg with RemoveUserFromOrg you will see they only differ with one check. If we remove ourselves from org we don't need to check if we are admins.

The other option was to have one function with an additional boolean parameter:
RemoveUserFromOrg(pk *ecdsa.PublicKey, requireAdmin bool)

Since boolean arguments are bad for code readability (see example below), I decided to go with the first one.

func (a *Api) LeaveCommunity() {
    community.RemoveOurselvesFromOrg(m.identity.pubKey)
}
func (a *Api) RemoveUserFromOrg(pubKey *PubKey) {
    community.RemoveUserFromOrg(pubKey)
}

vs

func (a *Api) LeaveCommunity() {
    community.RemoveUserFromOrg(m.identity.pubKey, false)
}
func (a *Api) RemoveUserFromOrg(pubKey *PubKey) {
    community.RemoveUserFromOrg(pubKey, true)
}