Develop #131
Merged
Develop #131
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
Added IDE run configurations for various services including Admin-Service, Authentication-Service, Chess-Service, and others with development profiles. Updated JRE version from OpenJDK 21 to OpenJDK 23 for test configurations. This setup ensures consistent and efficient local development.
This commit introduces new Writerside documentation files for various services, deployment, and system overview. It also updates the `Diagram.drawio` file to reflect recent service changes and connections for better clarity.
Introduced new topics: Development and Liquibase, with details on setup and usage. Enhanced existing content in New Deployment and Database with links for better navigation. Updated the table of contents to reflect these changes and added a placeholder for publishing documentation instructions.
Expanded the documentation to include detailed deployment steps, SSL management, and automated database backups. Added sections for new libraries and reorganized the structure for better clarity. Updated build profiles and linked relevant topics to improve usability.
Replaced Liquibase with Flyway in the docs and updated related references. Added new structured documentation for various topics, including Docker, Maven, and libraries. Streamlined outdated content in README and reorganized table of contents for better navigation.
Update documentation structure and Writerside docs
Updated PrimeNG and related dependencies to version 19.0.2 for improved compatibility and features. Removed redundant code and temporary search functionality from the chess settings component, simplifying the implementation. This also updates related templates for better maintainability.
The fideId has been removed from the Person model and migrated to the Account model under the FIDE platform. This change ensures better separation of concerns by associating platform-specific information with accounts. Additionally, the OVER_THE_BOARD platform was replaced with FIDE across the application.
Remove unused `url` field from Account conversion logic and update AccountDto and AccountConverter accordingly. Correct SQL migration to check `fide_id` instead of `platform_id` for adding FIDE accounts. Includes a placeholder for generating `url` in AccountConverter.
Introduce `internalComment` to event-related models, DTOs, and database schema to enable admin-specific notes in events. Enhance authentication processes by integrating server exchange matchers, replacing reactive authentication methods, and updating event service logic to handle permissions for admin users. Remove `fideId` from person forms as part of cleanup.
TODO needs correction and testing. Removed reactive Spring security setup in favor of servlet-based security with `ServletAuthenticationFilter`. Updated API implementations to replace `WebClient` with `RestClient`, improving handling of requests and responses. Removed unused legacy classes and configurations specific to the reactive setup.
Replaced inline form data builder with DTO classes for Fitbit and Spotify OAuth requests to improve maintainability and clarity. Added new DTOs for token requests, ensuring consistent structure and easier extensibility. Also introduced support for a new "FREESTYLE" chess platform across services and database schema.
This commit removes unused or unnecessary fields such as `fideId` and `url` from various test data models and test cases. The cleanup ensures that tests only include essential fields, improving maintainability and clarity of the codebase.
Replaced WebTestClient with MockMvc for consistency across test cases and improved maintainability. Added Testcontainers configuration for better test isolation and removed redundant or outdated configurations. Simplified and parameterized repeated test logic for improved readability and efficiency.
Introduce `FlywayClearDatabaseJunitExtension` to clean and migrate databases before each test across all services. Replace `DirtiesContext` with this extension in `TestcontainersConfiguration` and ensure proper Flyway configuration with test properties. This enhances test isolation and improves database handling consistency.
Enable `build`, `defaults`, `env`, `java`, `os`, `process`, and `ssl` under the `info` section across multiple services’ configuration files. This enhances visibility of system and build metadata in the various service configurations. Also updates the authentication manager to handle multiple authentications properly.
This update removes all dependencies and logic related to auth cookies, including their handling in interceptors, back-end configurations, and logout endpoints. Additionally, it introduces a microservice overview implementation with new components, services, and models for managing and displaying application statuses in the web interface.
Replaced specific actuator request matchers with a global rule requiring ADMIN_SERVICE authority for all requests. This simplifies configuration and ensures consistent access control across the application.
…programming remove-f-webflux-and-reactive-programming
Reactive Spring Security classes and annotations were removed across services to streamline configurations. Frontend code was updated to enhance the application's structure and data visualization, including detailed accordion views for microservice statuses.
Standardized import formatting across multiple files to align with the project style guide. Removed unused imports and optimized the organization within various components and services to improve readability and maintainability.
Added a new topic on using Testcontainers in integration tests to address limitations with h2 and better replicate production environments. Updated SonarQube workflow with a note on potential Testcontainers Cloud usage, though no explicit changes to the workflow yet.
Replaced manual participant management logic with reactive `rxResource` for better efficiency and maintainability. Simplified participant data flow in multiple components and updated binding in HTML templates. Removed redundant `participants` property from backend DTOs.
This commit integrates the ChessPlatform attribute into events, ensuring it is persisted, validated, and exposed across both the backend and frontend layers. It also replaces FIDE-specific links with dynamic account URLs based on the platform (e.g., Chess.com, Lichess). These changes enhance platform-specific functionality and improve user experience.
Introduce a `platform` field to chess events with validation in forms and UI integration. Simplify event management by replacing lazy loading with direct event fetching. Removed unused or redundant code for cleaner implementation.
Replaced the participant table with a grid of reusable player card components for improved readability and modularity. Introduced a new `ChessPlayerCardComponent` to encapsulate player details and actions. Adjusted associated logic and templates to support this change.
Introduced gender icons in various components using the refactored `EventIconPipe`. Added sorting functionality for chess event participants by different attributes such as firstname, lastname, federation, and birthday. Updated templates and styling to accommodate these changes.
126 refactor database schema
Updated sorting options to use localized translation keys for better i18n support. Adjusted the backend API by removing an unnecessary `principal` parameter in the `getEventCategoriesWithEvents` method. Also restructured JSON files for sorting translations and added missing entries.
MichiBaum
added
documentation
|
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.
resolves #126
many bugfixes & refactorings