Develop #170
Merged
Develop #170
Conversation
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
- Updated copyright year to 2025 in relevant files. - Refactored `Order` and `User` classes for property initialization. - Modified `Program.cs` to adjust using directives and MongoDB service registration. - Commented out `InitializeDatabasesAsync` method call in `Program.cs`. - Added methods for correlation context logging and error handling in `Extensions.cs`. - Renamed `MongoDbOptions` to `YarpMongoDbOptions` and updated properties. - Updated `MongodbConfigProvider` to use `YarpMongoDbOptions`. - Adjusted `Startup` class to use new MongoDB registration method. - Enhanced documentation comments in `Extensions.cs`. - Updated README to reflect library support and versioning changes, including .NET 8 transition.
Updated method comments in the `Extensions`, `IInitializer`, `BuilderExtensions`, `MongoDb`, `ErrorHandlerMiddleware`, `RedisSettingsBuilder`, `IRedisOptionsBuilder`, `Pagination`, and `JsonOutputFormatter` classes for clarity and consistency. Enhanced readability by standardizing return descriptions and providing clearer context for method parameters and functionality. Overall, these changes aim to make the documentation more informative for developers and users.
…ks across multiple components
Enhanced the TokenProvider class with a summary comment that describes its functionality for generating JWT tokens, including details on configurable settings for issuer, audience, key, and expiration time. This update aims to improve code clarity and assist developers in understanding the class's purpose and usage.
- Updated various NuGet package references across multiple projects to newer versions for compatibility and access to the latest features. - Enhanced `README_NUGET.md` for clarity, added a link to the GitHub repository, and included a change log and breaking changes. - Improved documentation comments in `Extensions.cs` for better understanding of method parameters. - Removed outdated package references and replaced them with updated counterparts, particularly for ASP.NET Core and OpenTelemetry. - Cleaned up project files by removing unnecessary references and aligning all projects with the latest versioning standards. - Adjusted versioning of several packages to specific stable versions instead of wildcards. - Updated `Directory.Build.props` to reflect the new version of `Roslynator.Analyzers`. - Streamlined `README_NUGET.md` by removing deprecated comments and sections.
- Updated Makefile to modify project paths and add Docker build/push targets. - Revised README.md to include new build scripts and Kubernetes deployment instructions. - Changed Prometheus configuration to reflect new service names. - Renamed identity service Dockerfile in the solution file. - Updated appsettings.json and related files for consistent issuer references. - Modified docker-compose files to change service names and update build contexts. - Added new Kubernetes YAML files for deploying apigateway, identities, products, orders, and notifications services. - Updated local.env with new image version and environment variables. - Added/modified build-images.sh, deploy-k8s.sh, and push-images.sh for automation. - Changed index.html title from "Genocs SignalR" to "Genocs Notifications". - Updated README_NUGET.md for monitoring issuer settings. - Ensured consistency in appsettings.Containers.json and other configuration files.
- Added notes on console and infrastructure setup in README.md. - Introduced "Aspire Integration" section for future plans. - Refined logger settings with more detailed path exclusions. - Clarified instructions for building and pushing Docker images. - Updated project contributors' sections with new links and images. - Enhanced logging configuration in Startup.cs for improved granularity.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request appears to be a large-scale maintenance update that includes package upgrades, refactoring improvements, and copyright year updates for 2025. The changes span across multiple components of the Genocs library.
- Update package dependencies and NuGet package versions across the codebase
- Refactor code to use modern C# syntax patterns (primary constructors, collection expressions)
- Update copyright notices and documentation for 2025
Reviewed Changes
Copilot reviewed 67 out of 67 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| src/apps/orders/Orders.WebApi/Orders.WebApi.csproj | Pin Genocs package versions from wildcard to specific 7.2.3 |
| src/apps/apigateway/APIGateway.WebApi/Startup.cs | Replace deprecated MongoDB registration methods |
| src/apps/apigateway/APIGateway.WebApi/Providers/MongodbConfigProvider.cs | Update MongoDB options type reference |
| src/apps/apigateway/APIGateway.WebApi/Configurations/MongoDbOptions.cs | Refactor to inherit from base MongoDbOptions |
| src/Genocs.WebApi/Formatters/JsonOutputFormatter.cs | Convert to primary constructor pattern |
| src/Genocs.WebApi/Formatters/JsonInputFormatter.cs | Update to use collection expression syntax |
| src/Genocs.WebApi/Extensions.cs | Add comprehensive XML documentation |
| Multiple project files | Update NuGet package versions and dependencies |
| Various source files | Add documentation, refactor syntax, and modernize code patterns |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
No description provided.