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

Do not allow null in observations columns #693

Open
petmongrels opened this issue Feb 19, 2024 · 1 comment
Open

Do not allow null in observations columns #693

petmongrels opened this issue Feb 19, 2024 · 1 comment

Comments

@petmongrels
Copy link
Contributor

petmongrels commented Feb 19, 2024

Having null values requires us to check for null instead of using ObservationCollection object. We should keep the values as {} and make it default value. Not making it default may cause the server to break, if null.

Analysis:

  • Was created as result of https://github.com/avniproject/data-fixes/issues/2
  • The data has become null because of data loss issue - date matches with created_date_time
  • If null can be updated on column with default value then not null null constraint also needed
  • Also need to check in setting place this check need to be made
@mahalakshme
Copy link
Contributor

mahalakshme commented Feb 20, 2024

@petmongrels this was identified as part of the ticket you worked on yesterday?

the entry that dealt here looks like was created by the IHMP data loss issue(1st time issue)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New Issues
Development

No branches or pull requests

2 participants