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.

Sign up for GitHub

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

Jump to bottom

Avoid coupling classes with DateTime.Now #58

Closed
MrDave1999 opened this issue Sep 30, 2022 · 0 comments · Fixed by #130
Closed

Avoid coupling classes with DateTime.Now #58

MrDave1999 opened this issue Sep 30, 2022 · 0 comments · Fixed by #130
Labels
refactor Refactoring existing code, to make it more readable, cleaner or more efficient

Comments

@MrDave1999
Copy link
Member

MrDave1999 commented Sep 30, 2022 •
edited
Loading

Currently the IDateTimeProvider interface (which is in the Helpers module) has been added for this problem, but some modules still need to be refactored to depend on the interface to facilitate unit testing.
@MrDave1999 MrDave1999 added the refactor Refactoring existing code, to make it more readable, cleaner or more efficient label Sep 30, 2022
@MrDave1999 MrDave1999 changed the title Avoid coupling classes with DateTime.Now Avoid coupling classes with DateTime.Now Sep 30, 2022
@MrDave1999 MrDave1999 mentioned this issue Jan 5, 2023
Merged
@MrDave1999 MrDave1999 added this to the v1.0.0 milestone Jan 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactor Refactoring existing code, to make it more readable, cleaner or more efficient
Projects
No open projects
DentallApp (first stage)
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Avoid coupling classes with DateTime.Now DentallApp/back-end
1 participant
@MrDave1999