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

Closes #2592 Embargo badge is not show/does not disappear immediately… #2594

Merged
merged 1 commit into from
Nov 28, 2024

Conversation

lbownik
Copy link
Contributor

@lbownik lbownik commented Nov 27, 2024

… after setting up/lifting an embargo

@lbownik lbownik requested a review from diasf November 27, 2024 15:23
@@ -235,6 +239,7 @@ private Dataset updateDatasetEmbargoDate(Dataset dataset, Date embargoDate) thro
dataset.setEmbargoDate(embargoDate);
dataset.setLastChangeForExporterTime(Date.from(Instant.now(Clock.systemDefaultZone())));
dataset = datasetDao.mergeAndFlush(dataset);
this.indexService.indexDataset(dataset, false);

solrIndexService.indexPermissionsForDatasetWithDataFiles(dataset);
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this can be removed, it should already be done by indexService.indexDataset.

@diasf diasf assigned lbownik and unassigned diasf Nov 28, 2024
@lbownik lbownik force-pushed the 2592_embargo_badge_is_not_shown branch from 9a449b6 to 7329d6c Compare November 28, 2024 11:36
@lbownik lbownik merged commit 0ecb1bd into develop Nov 28, 2024
1 check passed
@lbownik lbownik deleted the 2592_embargo_badge_is_not_shown branch November 28, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants