Skip to content

IUS-2613 embargo helper unit tests#201

Open
rdlebeau wants to merge 2 commits intomainfrom
IUS-2613_embargo_helper_unit_tests
Open

IUS-2613 embargo helper unit tests#201
rdlebeau wants to merge 2 commits intomainfrom
IUS-2613_embargo_helper_unit_tests

Conversation

@rdlebeau
Copy link
Contributor

@rdlebeau rdlebeau commented Oct 29, 2025 •
edited
Loading

IUS-2613 DataCORE EmbargoHelper corrections

  • Spelling correction - changed exipration_days to expiration_days
  • Removed unused parameter from days_to_embargo_release_date function
  • Removed empty function warn_deactivate_embargo_email

@rdlebeau rdlebeau marked this pull request as draft October 29, 2025 19:29
@rdlebeau rdlebeau force-pushed the IUS-2613_embargo_helper_unit_tests branch from bf1656f to a08f426 Compare October 31, 2025 15:01
@rdlebeau rdlebeau force-pushed the IUS-2613_embargo_helper_unit_tests branch from a08f426 to b0608d9 Compare November 14, 2025 20:45
@rdlebeau rdlebeau changed the title DRAFT IUS-2613 embargo helper unit tests IUS-2613 embargo helper unit tests Nov 14, 2025
@rdlebeau rdlebeau requested a review from aploshay November 14, 2025 20:56
@rdlebeau rdlebeau marked this pull request as ready for review November 14, 2025 20:57
@rdlebeau rdlebeau force-pushed the IUS-2613_embargo_helper_unit_tests branch from b0608d9 to fc094b6 Compare February 6, 2026 20:07
@rdlebeau rdlebeau force-pushed the IUS-2613_embargo_helper_unit_tests branch from fc094b6 to 91f6c37 Compare February 6, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aploshay aploshay Awaiting requested review from aploshay

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rdlebeau