Dev#72
Closed
abbasajorlou1371 wants to merge 13 commits into
Closed
Conversation
…ated files - Deleted the Node.js WebSocket gateway implementation, including its configuration, Dockerfile, and package files, to streamline the project towards a Go-based architecture. - Updated documentation and configuration files to reflect the removal of the WebSocket gateway service. - Adjusted `.gitignore` and `CONTRIBUTING.md` to remove references to the Node.js environment.
…uration - Introduced a new CORS preflight route for the `/api` endpoint to handle OPTIONS requests without rate-limiting, improving compatibility with CORS requests. - Enhanced the existing rate-limiting plugin configuration for the `api-routes` to utilize Redis for better performance and scalability. - Updated CORS plugin configuration to include additional exposed headers and refined origin handling for local development. - Added error handling in the `GetFamily` method to return a 404 status when a family is not found. Co-authored-by: Cursor <cursoragent@cursor.com>
… update gRPC configuration - Added an HTTP server to the calendar service to handle RESTful requests for calendar events, including endpoints for retrieving events, filtering by date range, and getting event details. - Updated the Docker and Kubernetes configurations to expose the new HTTP port (8060) and ensure proper service dependencies. - Removed the calendar service address from the grpc-gateway configuration, as it now communicates via HTTP. - Enhanced health checks and added middleware for authentication in the HTTP layer. - Updated environment variables and configuration files to reflect the new service architecture.
…itecture - Added an HTTP server to the features service, enabling RESTful access to features-related endpoints. - Updated Docker and Kubernetes configurations to expose the new HTTP port (8062) and ensure proper service dependencies. - Removed gRPC communication for the features service from the grpc-gateway configuration, transitioning to HTTP. - Enhanced health checks and added middleware for authentication in the HTTP layer. - Updated environment variables and configuration files to reflect the new service architecture. - Refactored existing handlers to support the new HTTP routes and functionality.
…iple services - Upgraded `google.golang.org/grpc` from v1.79.3 to v1.82.1 in various services to ensure compatibility with the latest features and security updates. - Updated `google.golang.org/genproto/googleapis/rpc` to a newer version across services to maintain consistency and support for the latest gRPC functionalities. - Adjusted `go.mod` and `go.sum` files in services including auth-service, calendar-service, commercial-service, dynasty-service, features-service, financial-service, grpc-gateway, and levels-service to reflect these changes. - Ensured all indirect dependencies are also updated to their latest versions for improved stability and performance.
… up imports - Deleted the `writeFieldValidationError` function from `pagination.go` as it was not utilized in the codebase. - Removed unnecessary `encoding/json` import from both `pagination.go` and `query_params.go` files to streamline the code. - This cleanup enhances code readability and reduces potential confusion regarding unused functions.
- Modified the Makefile to streamline test commands by removing `test-integration` and `test-golden` targets, consolidating them into `test-all`. - Updated the help section to reflect changes in test commands and clarify the purpose of `test` as an alias for `test-all`. - Adjusted README documentation to align with the new test structure and removed references to integration and golden tests. - Deleted obsolete integration and load test workflows from GitHub Actions to simplify CI/CD processes. - Cleaned up the features service testing documentation to focus on unit and service tests, removing references to integration tests.
- Updated the CheckBalance method to use a raw balance string for asset types, improving clarity and error handling. - Enhanced the parseWalletString function to support parsing of compact number formats, utilizing a new helper function for better accuracy. - Added error handling for invalid asset types and balance checks in the MarketplaceService, ensuring more robust error reporting. - Improved overall readability and maintainability of the balance checking logic across services.
…and transfer logic - Updated the Create method to ensure a single hourly profit record per feature, reassigning existing records and deleting duplicates. - Introduced a new oneFeaturePropertyJoin constant to streamline SQL joins, ensuring only one property row is returned per feature. - Refactored TransferProfitToNewOwner methods to guarantee a single profit row per feature_id, improving clarity and maintainability. - Adjusted service methods to utilize the updated transfer logic, ensuring accurate profit transfers during feature purchases.
… summaries - Introduced `go-test-checklist.sh` to run Go tests and generate a markdown checklist for GITHUB_STEP_SUMMARY, detailing test results and failures. - Added `govulncheck-checklist.sh` to execute govulncheck and append vulnerability findings to the markdown summary. - Updated CI workflows to utilize these scripts for unit and integration tests, as well as security scans, enhancing reporting and visibility of test outcomes. - Implemented summary sections in the CI workflows to consolidate test results and vulnerability statuses for better clarity in the CI output.
…ed logging and output - Updated `go-test-checklist.sh` to separate standard output and error logs, providing clearer feedback on test results and failures. - Enhanced `govulncheck-checklist.sh` to limit log output size and improve error handling, ensuring concise summaries in GITHUB_STEP_SUMMARY. - Modified CI workflows to ensure all test suites run regardless of previous failures, improving overall visibility of test outcomes. - Adjusted summary sections in workflows to reflect package-level results and individual test failures, enhancing clarity in CI output.
…flows - Updated the integration test step to check for the presence of `go.mod` before execution, providing clearer feedback when tests cannot run. - Enhanced the output status handling for integration tests, ensuring accurate reporting in the CI summary. - Adjusted the unit test command to cover all relevant directories, improving test coverage. - Added test paths for various services in the services CI workflow, ensuring all relevant tests are included in the CI process.
…aries - Modified `go-test-checklist.sh` to allow toggling of summary writing based on the `CHECKLIST_WRITE_SUMMARY` environment variable, improving log compactness. - Adjusted CI workflow to set `CHECKLIST_WRITE_SUMMARY` to "false" for unit tests, ensuring that only essential information is included in the summary output.
Collaborator
|
1 |
Collaborator
|
restart test |
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
2 participants
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.