Merged
Conversation
Cleaned up `IMattermostClientExtensions.cs` by removing the `MattermostClientExtensions` class, its namespace, and unused using directives. Streamlined `FileDetails.cs` by removing unnecessary using directives and relocating the `FileResponse` class. Moved the `FileResponse` class to a new file (`FileResponse.cs`) with appropriate using directives and namespace for better organization.
Updated README.md example to use new MattermostClient login method. Updated package references in Mattermost.Tests.csproj and Mattermost.csproj. Refactored imports and namespaces in various files for better organization. Removed handling of the Unknown event in MattermostClient.cs.
Added OnEventReceived event in MattermostClient class, triggered on any event reception. Implemented its invocation in HandleResponseAsync with exception handling. Modified Json property in WebsocketMessage to return "{}" when Data is null or whitespace. Introduced WebSocketEventArgs class in Mattermost.Events namespace to encapsulate WebSocket event data.
The target framework for the `Mattermost.ConsoleTest.csproj` and `Mattermost.Tests.csproj` projects has been updated from .NET 8.0 to .NET 9.0. This change is reflected by modifying the `<TargetFramework>` element in both project files. +semver: major
Added XML documentation comments to the MattermostEvent enum and its members in MattermostEvent.cs. This includes a summary for the enum itself and each member, explaining the event types they represent. Removed the #pragma warning disable CS1591 directive to enable warnings for missing XML comments.
README.md reorganized for clarity: removed "Getting Started," updated "Installation," split "Usage" into subsections, and added examples for client methods. IMattermostClient.cs: added `CreatePostAsync`, marked `SendMessageAsync` as obsolete. MattermostClient.cs: `SendMessageAsync` now calls `CreatePostAsync` and is marked obsolete.
The method signature for `CreatePostAsync` in the `IMattermostClient` interface has been removed. This method was responsible for creating a post in a specified channel with optional parameters for message text, reply-to post ID, message priority, and attached files.
Replaced SendMessageAsync with CreatePostAsync across multiple files: - Updated OnMessageReceived event handler in Program.cs - Modified test case in MattermostClientTests.cs - Changed post construction method in PostBuilder.cs
a981d69 to
594795f
Compare
This file contains hidden or 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
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.
This PR adds "props" property to create and update posts api, implements new api for GetPostsForChannel, allows to upload fire by using Stream object