Skip to content

Conversation

@marioserrano09
Copy link
Contributor

This pull request introduces several updates to the SaaS module, including dependency version upgrades, changes to entity relationships, and the addition of new fields and services to enhance functionality. The changes are grouped into dependency updates, entity modifications, and service enhancements.

Dependency Updates:

  • Upgraded the parent and module versions in pom.xml files for api and core to 3.4.1 from 3.4.0, ensuring compatibility with the latest features and fixes. ([[1]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-978f907ee3c84eb4e8cc278641ab9dd89ad9e43f6a39b81803eeb5e7bff64426L29-R35), [[2]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-78e04ac41aa4c32ba3c532640022978c0b8ed25d646cd9f94257baa2cb2c8ea0L25-R28), [[3]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-78e04ac41aa4c32ba3c532640022978c0b8ed25d646cd9f94257baa2cb2c8ea0L52-R57))

Entity Modifications:

  • Updated relationships in several domain classes (Account, AccountCharge, AccountLog, AccountPayment, AccountReseller) from @OneToOne to @ManyToOne to better reflect the data model and improve scalability. ([[1]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-866f875e16ea394c7c94e94b7dc242d2d0b68bbebfe8c3db318e0f36f77094d5L81-R82), [[2]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-2bf400d9bef7f1be23edabd929900b9f6be4be2dd43cc269d9ae823733dd2140L32-R33), [[3]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-bb3e53578eee171b7de3dc50a07d7ecc49d0b9ed3fed469b6c8a701be5e5bf1fL37-R37), [[4]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-9f8c5b639cf683fd44d0581ae6102e70c803d636345a49fdcf235a4b11c80e23L46-R49), [[5]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-0120ce050109b9a718b63061657dff9c9158919a1ca9b5496be0c7f76cc05c75L47-R55))
  • Added new fields to Account and AccountReseller for reseller management, including resellerAgent, resellerComments, resellerInvoice, and resellerInvoiceDate in Account, and agents, startDate, endDate, and comments in AccountReseller. Corresponding getter and setter methods were also added. ([[1]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-866f875e16ea394c7c94e94b7dc242d2d0b68bbebfe8c3db318e0f36f77094d5R171-R182), [[2]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-0120ce050109b9a718b63061657dff9c9158919a1ca9b5496be0c7f76cc05c75L47-R55), [[3]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-0120ce050109b9a718b63061657dff9c9158919a1ca9b5496be0c7f76cc05c75R125-R156))
  • Introduced LocalDate import in Account for potential future use. ([sources/core/src/main/java/tools/dynamia/modules/saas/domain/Account.javaR43](https://github.com/dynamiatools/module-saas/pull/14/files#diff-866f875e16ea394c7c94e94b7dc242d2d0b68bbebfe8c3db318e0f36f77094d5R43))

Service Enhancements:

  • Added a new AccountResellerService interface to manage operations related to resellers and their agents, including methods for retrieving enabled resellers, agents, payments, and accounts. ([sources/core/src/main/java/tools/dynamia/modules/saas/services/AccountResellerService.javaR1-R52](https://github.com/dynamiatools/module-saas/pull/14/files#diff-7c999252bbca4942e4b404013b0973606fd9cb45b9fc47da1098b7564073322fR1-R52))
  • Enhanced the AccountService interface with detailed documentation and additional methods for account management, including retrieving accounts by various criteria, handling payments, and managing cache. ([[1]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-553b16b5bf67059728198dfc87da42156ea06fa4e2e73bf7d6dcd3aca4a43da3L1), [[2]](https://github.com/dynamiatools/module-saas/pull/14/files#diff-553b16b5bf67059728198dfc87da42156ea06fa4e2e73bf7d6dcd3aca4a43da3L21-R236))

Copilot AI review requested due to automatic review settings July 31, 2025 20:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request enhances the SaaS module to support account reseller agents functionality while upgrading all module versions from 3.4.0 to 3.4.1. The changes introduce a new agent-based reseller management system with improved entity relationships and comprehensive documentation.

  • Version upgrade across all modules to 3.4.1 for compatibility and latest features
  • Entity relationship improvements from @OnetoOne to @manytoone for better scalability and data modeling
  • Addition of reseller agent functionality with new domain entities, services, and UI components

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
sources/core/src/main/java/tools/dynamia/modules/saas/domain/Account.java Added reseller agent fields and relationship mappings
sources/core/src/main/java/tools/dynamia/modules/saas/domain/AccountReseller.java Enhanced with agent management capabilities and date tracking
sources/core/src/main/java/tools/dynamia/modules/saas/services/AccountResellerService.java New service interface for reseller and agent operations
sources/core/src/main/java/tools/dynamia/modules/saas/services/impl/AccountResellerServiceImpl.java Implementation of reseller service with CRUD operations
sources/ui/src/main/java/tools/dynamia/modules/saas/ui/customizers/AccountFormCustomizer.java UI customizer for dynamic reseller/agent selection
Multiple domain entities Updated relationships from @OnetoOne to @manytoone for improved data modeling
Multiple pom.xml files Version updates from 3.4.0 to 3.4.1
UI descriptor files Added form and table configurations for reseller agents

@marioserrano09 marioserrano09 merged commit c3d98ed into master Jul 31, 2025
1 check passed
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.

2 participants