Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

fix defaults for time columns in user table (with orm) #1190

Merged
merged 2 commits into from
Jun 9, 2023
Merged

fix defaults for time columns in user table (with orm) #1190

merged 2 commits into from
Jun 9, 2023

Conversation

melange396
Copy link
Collaborator

defaults for date columns last_time_used and created in the User table were statically set to whenever that code was first evaluated. this makes them dynamic.

@melange396 melange396 requested a review from dmytrotsko June 7, 2023 19:23
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@melange396 melange396 merged commit 2e4d309 into dev Jun 9, 2023
@melange396 melange396 deleted the user_default_dates branch June 9, 2023 22:37
@melange396 melange396 mentioned this pull request Jun 13, 2023
Merged
@melange396 melange396 mentioned this pull request Jun 23, 2023
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@dmytrotsko dmytrotsko dmytrotsko approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@melange396 @dmytrotsko