Skip to content

[ENG-11214] - Fix missing registrations from Internet Archive backups#11769

Open
Vlad0n20 wants to merge 1 commit into
CenterForOpenScience:feature/pbs-26-13from
Vlad0n20:fix/ENG-11214
Open

[ENG-11214] - Fix missing registrations from Internet Archive backups#11769
Vlad0n20 wants to merge 1 commit into
CenterForOpenScience:feature/pbs-26-13from
Vlad0n20:fix/ENG-11214

Conversation

@Vlad0n20

@Vlad0n20 Vlad0n20 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Ticket

Purpose

Changes

Side Effects

QE Notes

CE Notes

Documentation

@adlius adlius changed the base branch from feature/pbs-26-9 to feature/pbs-26-13 July 6, 2026 15:37

@adlius adlius left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

One improvement.

).exclude(
moderation_state='withdrawn',
)
).distinct()

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We probably won't need .distinct() here. Also it seems adding distinct() here would break the archive_registrations_on_IA() script because it uses this find_ia_backlog() queryset and adds order_by('?').

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