Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull Request: Migrate Project to .NET 8 and Improve Yield Calculation #7

Merged
merged 16 commits into from
Aug 13, 2024

Conversation

almeidaalex
Copy link
Owner

Pull Request: Migrate Project to .NET 8 and Improve Yield Calculation

Summary

This pull request includes significant updates to migrate the project to .NET 8, improve the yield calculation logic, and fix several issues related to Entity Framework (EF) mappings. The changes also include updates to the documentation and configuration for better development experience.

Changes

  • Migration to .NET 8:

    • Updated the project to target .NET 8.
    • Migrated the domain to .NET 8.0.
  • Yield Calculation Improvements:

    • Refactored the yield calculator to only calculate the yield without setting values directly on the account.
    • Updated tests to compare values returned from YieldService.
  • Entity Framework Fixes:

    • Fixed relationship mapping by adding account to AccountOperation.
    • Ensured new operations are not added when setting a new yield.
  • Documentation and Configuration:

    • Updated the .NET version in the README.
    • Created configuration to enable debugging in VS Code.
    • Removed unused namespaces for cleaner code.

Additional Notes

  • All changes have been tested and verified in the staging environment.
  • Documentation has been updated to reflect the new features and changes.
  • The project is now fully compatible with .NET 8, and the yield calculation logic is more robust and maintainable.

Please review the changes and provide feedback. Thank you!

@almeidaalex almeidaalex linked an issue Aug 13, 2024 that may be closed by this pull request
@almeidaalex almeidaalex merged commit c4f69b1 into main Aug 13, 2024
1 check passed
@almeidaalex almeidaalex deleted the 6-migrar-para-net-8 branch August 13, 2024 09:00
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.

Migrar para .NET 8
1 participant