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

Remove _since and favour created #1747

Merged
merged 1 commit into from
Sep 4, 2020

Conversation

Martii
Copy link
Member

@Martii Martii commented Sep 4, 2020

  • _since has a precision issue and possible extended time failure due to lack of bits
  • Fix a few timing issues
  • Best not to set default date in groups model as it can cover up errors and was doing it twice
  • Some identifier symmetry
  • All models should have base created with the exception of removed since once gone that's technically creation of nothingness.
  • Bump package version since older backups won't be valid after migration

Applies to #349

* `_since` has a precision issue and possible extended time failure due to lack of bits
* Fix a few timing issues
* Best not to set default date in groups model as it can cover up errors and was doing it twice
* Some identifier symmetry
* All models should have base `created` with the exception of `removed` since once gone that's technically creation of nothingness.
* Bump package version since older backups won't be valid after migration

Applies to OpenUserJS#349
@Martii Martii added bug You've guessed it... this means a bug is reported. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. DB Pertains inclusively to the Database operations. migration Use this to indicate that it may apply to an existing or announced migration. labels Sep 4, 2020
@Martii Martii merged commit b612274 into OpenUserJS:master Sep 4, 2020
@Martii Martii deleted the Issue-349createdFixes branch September 4, 2020 13:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug You've guessed it... this means a bug is reported. DB Pertains inclusively to the Database operations. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. migration Use this to indicate that it may apply to an existing or announced migration.
Development

Successfully merging this pull request may close these issues.

1 participant