Add unfollow and unblock methods #34

glyph · 2022-03-01T12:24:19Z

glyph commented

2022-03-01 12:24:19 +00:00

Since I'm currently adding the ability to unfollow or unblock a peer in Peachland, this seemed like a good time to add convenience methods.

I noticed that set_relationship was being unnecessarily noisy, in the sense that it was publishing values that were not technically required (ie. blocking: false when calling follow(peer_id)). I have updated that method to take Option<bool> for following and blocking, instead of simply bool.

unblock(peer_id)

This is a convenience method to publish a contact message with blocking: false.

unfollow(peer_id)

This is a convenience method to publish a contact message with following: false.

Since I'm currently adding the ability to unfollow or unblock a peer in Peachland, this seemed like a good time to add convenience methods. I noticed that `set_relationship` was being unnecessarily noisy, in the sense that it was publishing values that were not technically required (ie. `blocking: false` when calling `follow(peer_id)`). I have updated that method to take `Option<bool>` for `following` and `blocking`, instead of simply `bool`. `unblock(peer_id)` > This is a convenience method to publish a contact message with blocking: `false`. `unfollow(peer_id)` > This is a convenience method to publish a contact message with following: `false`.

glyph added 1 commit 2022-03-01 12:24:20 +00:00

6508aeb3ea add unfollow and unblock methods

glyph added 1 commit 2022-03-01 13:42:54 +00:00

e8294241ec update set_relationship parameter types

glyph merged commit 9ad38fb0e8 into main

2022-03-03 08:17:07 +00:00

glyph deleted branch unfollow_and_unblock

2022-03-03 08:17:08 +00:00

glyph referenced this issue from a commit

2022-03-03 08:17:08 +00:00

Merge pull request 'Add unfollow and unblock methods' (#34) from unfollow_and_unblock into main

Sign in to join this conversation.