Skip to content

Commit

Permalink
Copy the collection contents by default when copying a collection
Browse files Browse the repository at this point in the history
That got left out when creating the new multi history view.
Fixes #16716 and
#16619.
  • Loading branch information
mvdbeek committed Sep 21, 2023
1 parent 1eabd67 commit 9816c26
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions client/src/schema/schema.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2446,7 +2446,7 @@ export interface components {
/**
* Copy Elements
* @description If the source is a collection, whether to copy child HDAs into the target history as well, defaults to False but this is less than ideal and may be changed in future releases.
* @default false
* @default true
*/
copy_elements?: boolean;
/**
Expand Down Expand Up @@ -2616,7 +2616,7 @@ export interface components {
/**
* Copy Elements
* @description Whether to create a copy of the source HDAs for the new collection.
* @default false
* @default true
*/
copy_elements?: boolean;
/**
Expand Down
2 changes: 1 addition & 1 deletion lib/galaxy/schema/schema.py
Original file line number Diff line number Diff line change
Expand Up @@ -1366,7 +1366,7 @@ class CreateNewCollectionPayload(Model):
description="Whether to mark the original HDAs as hidden.",
)
copy_elements: Optional[bool] = Field(
default=False,
default=True,
title="Copy Elements",
description="Whether to create a copy of the source HDAs for the new collection.",
)
Expand Down
2 changes: 1 addition & 1 deletion lib/galaxy/webapps/galaxy/services/history_contents.py
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,7 @@ class CreateHistoryContentPayloadFromCollection(CreateHistoryContentPayloadFromC
description="TODO",
)
copy_elements: Optional[bool] = Field(
default=False,
default=True,
title="Copy Elements",
description=(
"If the source is a collection, whether to copy child HDAs into the target "
Expand Down

0 comments on commit 9816c26

Please sign in to comment.