Skip to content

Conversation

crimsonred
Copy link
Contributor

Message Actions
Fetch with Message Actions

@crimsonred crimsonred requested a review from davidnub November 21, 2019 01:31
@PubNubBot
Copy link

Codacy Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- PubNubUnity/Assets/PubNub/EndPoints/MessageActions/RemoveMessageActions.cs  1
- PubNubUnity/Assets/PubNub/Helpers/MessageActionsHelpers.cs  5
- PubNubUnity/Assets/PubNub/Workers/SubscriptionWorker.cs  2
- PubNubUnity/Assets/PubNub/Models/Consumer/MessageActions/PNMessageActionsResult.cs  1
- PubNubUnity/Assets/PubNub/Models/Consumer/PubSub/PNMessageActionsEventResult.cs  1
- PubNubUnity/Assets/PubNub/Builders/MessageActions/AddMessageActionsRequestBuilder.cs  4
- PubNubUnity/Assets/PubNub/Builders/MessageActions/RemoveMessageActionsRequestBuilder.cs  2
- PubNubUnity/Assets/PubNub/Builders/MessageActions/GetMessageActionsRequestBuilder.cs  6
- PubNubUnity/Assets/PubNub/EndPoints/MessageActions/GetMessageActions.cs  1
- PubNubUnity/Assets/PubNub/Enum/PNMessageActionEnums.cs  3
- PubNubUnity/Assets/PubNub/Managers/QueueManager.cs  3
- PubNubUnity/Assets/PubNub/Models/Consumer/MessageActions/PNGetMessageActionsResult.cs  1
- PubNubUnity/Assets/PubNub/EndPoints/MessageActions/AddMessageActions.cs  1
- PubNubUnity/Assets/PubNub/PubNubUnity/PubNubUnity.cs  1
         

Clones added
============
- PubNubUnity/Assets/PubNub/Builders/MessageActions/AddMessageActionsRequestBuilder.cs  4
- PubNubUnity/Assets/PubNub/Builders/Objects/CreateUserRequestBuilder.cs  3
- PubNubUnity/Assets/PubNub/Builders/MessageActions/GetMessageActionsRequestBuilder.cs  2
         

Clones removed
==============
+ PubNubUnity/Assets/PubNub/Builders/History/FetchMessagesRequestBuilder.cs  -1
         

See the complete overview on Codacy

Copy link

@davidnub davidnub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 👍

@crimsonred crimsonred merged commit 7d74a9a into master Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants