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

Change content date defaults to None from '' #361

Merged
merged 1 commit into from
Jun 9, 2023

Conversation

k-burt-uch
Copy link
Contributor

New Features

Breaking Changes

Bug Fixes

  • Indexd records no longer return empty strings for content_updated_date and content_updated_time when not provided.

Improvements

Dependency updates

Deployment changes

@github-actions
Copy link

github-actions bot commented Jun 5, 2023

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

MaribelleHGomez
MaribelleHGomez previously approved these changes Jun 7, 2023
Copy link
Contributor

@MaribelleHGomez MaribelleHGomez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm :)

@k-burt-uch k-burt-uch force-pushed the fix/content-date-default-to-none branch from e32062d to a498afa Compare June 9, 2023 20:16
Copy link
Contributor

@BinamB BinamB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@k-burt-uch k-burt-uch merged commit c6c8a73 into master Jun 9, 2023
4 checks passed
@k-burt-uch k-burt-uch deleted the fix/content-date-default-to-none branch June 9, 2023 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants