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

[tools][CouchDB_MRI_Importer] Fix date in MRI data #8851

Merged
merged 4 commits into from
Aug 16, 2023

Conversation

charlottesce
Copy link
Contributor

Brief summary of changes

The first change in this PR is changing the order of arguments in the function call to join(), as I got the following error while running the script initially:
Screenshot 2023-08-01 135343

Then, it fixes the date format {total number of seconds} to a proper YYYY-mm-dd format, for "FileInsert{date}" variables from
Screenshot 2023-08-01 135127
to:
Screenshot 2023-08-01 134933

Testing instructions (if applicable)

  1. Run the CouchDB_Import_MRI.php script
  2. Make sure the variables starting with FileInsert{date} e.g. for mri_data, have a correct date format

Note

@charlottesce charlottesce added the Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS label Aug 1, 2023
@driusan driusan merged commit 6705373 into aces:24.1-release Aug 16, 2023
9 checks passed
@ridz1208 ridz1208 added this to the 24.1.5 milestone Nov 9, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants