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

Fix/6185 typeorm fixes #593

Merged
merged 6 commits into from
May 21, 2024
Merged

Fix/6185 typeorm fixes #593

merged 6 commits into from
May 21, 2024

Conversation

maxdiebold-erg
Copy link
Contributor

Main Changes

  • Added the new custom repositories to the exports of each module.
  • Switched to accessing repositories through module imports rather than including repositories from other modules in the TypeOrmModule.forFeature array.
  • Fixed an issue where the QaSuppDataWorkspaceModule was incorrectly being imported from qa-monitor-plan.module.ts. This issue seems to have been present already, but was found during TypeORM regression testing.
  • Fixed timestamp issues with several services, which were attempting to save a date string (toLocaleDateString) to a timestamp field.

Copy link

sonarcloud bot commented May 17, 2024

@mark-hayward-erg mark-hayward-erg merged commit 9899c3f into develop May 21, 2024
2 checks 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.

None yet

2 participants