Add support for relationships #284
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds Support for: - Sending friend request. - Accepting friend request. - Getting all the relationships. - Getting all the mutual friends with another user. - Blocking a user. **Note:** - Bot accounts are not allowed to access the endpoint. - May close bwmarrin#150
iopred
reviewed
Nov 9, 2016
| @@ -381,7 +381,7 @@ type Ready struct { | |||
| // A Relationship between the logged in user and Relationship.User | |||
| type Relationship struct { | |||
| User *User `json:"user"` | |||
| Type int `json:"type"` // 1 = friend, 2 = blocked, 3 = incoming friend req, 4 = sent friend req | |||
| Type uint8 `json:"type"` // 1 = friend, 2 = blocked, 3 = incoming friend req, 4 = sent friend req | |||
There was a problem hiding this comment.
Eh, might as well leave as int.
Changed the uint8 declarations to int.
iopred
reviewed
Nov 9, 2016
|
|
||
| // relationshipCreate creates a new relationship. (I.e. send or accept a friend request, block a user.) | ||
| // relationshipType : 1 = friend, 2 = blocked, 3 = incoming friend req, 4 = sent friend req | ||
| func (s *Session) relationshipCreate(userID string, relationshipType uint8) (err error) { |
There was a problem hiding this comment.
This uint8 broke, sorry about that.
There was a problem hiding this comment.
Mhmm. Forgot to run the tests.
Missed one instance of unint8 during previous push.
ErikMcClure
pushed a commit
to ErikMcClure/discordgo
that referenced
this pull request
Aug 4, 2020
* Add support for relationships Adds Support for: - Sending friend request. - Accepting friend request. - Getting all the relationships. - Getting all the mutual friends with another user. - Blocking a user. **Note:** - Bot accounts are not allowed to access the endpoint. - May close bwmarrin#150 * Implement requested changes Changed the uint8 declarations to int. * Change the missed unint8 declaration to int Missed one instance of unint8 during previous push.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds Support for:
Note: