Skip to content

Fix recursive collection download - #1764

Merged
cjmyers merged 1 commit into
masterfrom
fixRecursiveCollection
Oct 12, 2026
Merged

cjmyers merged 1 commit into
masterfrom
fixRecursiveCollection

Conversation

@cjmyers

@cjmyers cjmyers commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@cjmyers
cjmyers requested a review from danielfang97 October 10, 2026 01:53

@danielfang97 danielfang97 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.

Doesn't appear to be working. Here are the files I downloaded. The first one with (2) is the collection with required login turned on. The second file with (3) is the collection with required login turned off. The required login turned on collection is missing the recursive public collection info.

test_2_private_collection (2).xml
test_2_private_collection (3).xml

@danielfang97
danielfang97 self-requested a review October 12, 2026 17:00

@danielfang97 danielfang97 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.

Now working correctly, even with login required turned on.

@cjmyers
cjmyers merged commit ba72ef4 into master Oct 12, 2026
10 checks passed
@cjmyers
cjmyers deleted the fixRecursiveCollection branch October 12, 2026 19:00
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