Transfer requests don't remove unused assets
Merge pull request #890 from JamesGlover/prevent_transfer_requests_from_destroying_assets

Prevent transfer requests from destroying assets
Remove redundant code block
Merge pull request #891 from JamesGlover/prevent_transfer_requests_from_destroying_assets

Remove redundant code block
[#74677798] Update scopes to samples or ids

Rails scopes are smart enough to accept an array of
ActiveRecord objects or ids. This switch allows us
to reduce the memory footprint elsewhere.
[#74677798] Reduce memory requirements for manifests

Large manifests used the sample as the key in a hash.
This caused issues with larger manifests, as they'd
keep all samples in memory.

This is not required, and switching to sample.id
relieves the memory issues experienced.
CherrypickForPulldown hold->started transition
Merge pull request #893 from JamesGlover/74677798_ss33_-_resolve_ram_problem_for_large_sample_manifests

74677798 ss33   resolve ram problem for large sample manifests
Merge pull request #892 from JamesGlover/cherrypick_request_can_be_unheld

CherrypickForPulldown hold->started transition
Repair asset group scope
Merge pull request #894 from JamesGlover/fix_asset_group_search

Repair asset group scope
Merge pull request #895 from sanger/next_release

Fluidigm Fixes